Skip to content
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

Remove document formatting range registration and make formatting wor… #179

Merged
merged 1 commit into from
Sep 13, 2019

Conversation

JPinkney
Copy link
Contributor

@JPinkney JPinkney commented Sep 4, 2019

…k for all yaml files

Fixes: redhat-developer/vscode-yaml#220 which came about because documentRangeFormatter was being dynamically registered even though the feature is not actually supported in the language server.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 67.01% when pulling ca40f01 on fixDocumentFormattingOnPaste into 2be16f3 on master.

@JPinkney JPinkney merged commit 03af2f8 into master Sep 13, 2019
@evidolob evidolob deleted the fixDocumentFormattingOnPaste branch August 11, 2020 13:18
evidolob added a commit to evidolob/yaml-language-server that referenced this pull request Jan 12, 2021
evidolob added a commit that referenced this pull request Jan 18, 2021
* #179 add 'doDocumentOnTypeFormatting' implementation

Signed-off-by: Yevhen Vydolob <[email protected]>

* add tests

Signed-off-by: Yevhen Vydolob <[email protected]>

* add handler

Signed-off-by: Yevhen Vydolob <[email protected]>

* add missing trigger character

Signed-off-by: Yevhen Vydolob <[email protected]>

* Update tests

Signed-off-by: Yevhen Vydolob <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error on Paste - Range Formatting
2 participants