-
Notifications
You must be signed in to change notification settings - Fork 276
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
Feature Request: list-ordering #454
Comments
This is based on the key-ordering rule and behaves the same with regards to locales. Closes adrienverge#454
This is based on the key-ordering rule and behaves the same with regards to locales. Closes adrienverge#454
This is based on the key-ordering rule and behaves the same with regards to locales. Closes adrienverge#454
This is based on the key-ordering rule and behaves the same with regards to locales. Closes adrienverge#454
Hello, I'm not very favorable to such a change, because yamllint aims to lint the form, not the content:
Additionally, the way heterogenous item types (e.g. |
Any kind of list sorting needs to be based on content-specific policies. It isn't really feasible to implement such a thing within a linter. |
I do agree with the points @adrienverge raises, and is very much content-specific as @mwgamble notes. I do still have the need and cast an eye towards yamllint as the tool since it already has the yaml parsed into a DOM where I felt it might be an easy feature to add... Maybe we'll just write our own parser to implement this check for our context.. |
This is based on the key-ordering rule and behaves the same with regards to locales. Closes adrienverge#454
Hello, |
I agree that someone do not want to have list sorted, but this rule is similar to I think that it would be worthy to have it, disabled by default. And actually there is a PR for this feature, but what is the reason it was not merged? The lack of some test cases? I'm not sure but maybe @adrienverge you should write it explicitly, what should be added to the PR to be able to merge it. |
Hello, I tried to make it clear in my previous comment: #454 (comment) but I can try to write it differently. Yamllint aims to lint the form, not the content. The rule |
Similar to the
key-ordering
rule, I'd like to be able to enforce that lists are sorted.Pass:
Fail:
The text was updated successfully, but these errors were encountered: