-
Notifications
You must be signed in to change notification settings - Fork 278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle DocumentRangeFormattingRequest
#1806
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR, @MahdiBM. It’s looking great. I have left a few comments inline and I think they should also fix your issue that you don’t get any edits.
Ok I think the PR is looking good ... I'll mark as ready for review after I've added more tests. |
9bc9c65
to
9522225
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for implementing this @MahdiBM. Changes look great to me.
@swift-ci Please test |
@swift-ci Please test Windows |
Head branch was pushed to by a user without write access
6ec08b5
to
8f3253d
Compare
@swift-ci Please test |
@swift-ci Please test Windows |
partially resolves #1805