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

fix the default for keyorder #859

Merged

Conversation

gorkem
Copy link
Collaborator

@gorkem gorkem commented Mar 8, 2023

What does this PR do?

Fixes the default value for keyOrder setting

What issues does this PR fix or reference?

Is it tested? How?

Existing tests

Fixes the default value for keyordering
@coveralls
Copy link

coveralls commented Mar 8, 2023

Coverage Status

Coverage: 83.241%. Remained the same when pulling abf196f on gorkem:fix-keyorder-default into c657e66 on redhat-developer:main.

@msivasubramaniaan msivasubramaniaan merged commit c8fa80d into redhat-developer:main Mar 9, 2023
fnichol added a commit to fnichol/dotneovim that referenced this pull request Apr 14, 2023
The next release of yaml-language-server (post-1.12.0) will set the
`keyOrdering` setting to `false` by default.

References: redhat-developer/yaml-language-server#859

Signed-off-by: Fletcher Nichol <fnichol@nichol.ca>
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.

None yet

3 participants