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

Configuration file should be named .ls-lint.yaml #29

Closed
batisteo opened this issue Jul 27, 2020 · 5 comments
Closed

Configuration file should be named .ls-lint.yaml #29

batisteo opened this issue Jul 27, 2020 · 5 comments

Comments

@batisteo
Copy link

The extension .yaml should be preferred over .yml, in a similar way we use .html instead of .htm.

@loeffel-io
Copy link
Owner

Hey @batisteo,

thank you for your contribution!

I personally do prefer yml instead of yaml but i will take care about that in v2 by ls-lint -f .ls-lint.yaml.

@batisteo
Copy link
Author

Cool

From https://yaml.org/faq.html:

Is there an official extension for YAML files?
Please use ".yaml" when possible.

@id-ekaagr
Copy link

how about picking c as config file argument? ls-lint -c .ls-lint.yaml ls-lint --config .ls-lint.yaml and not ls-lint -f .ls-lint.yaml?

@yuhr
Copy link

yuhr commented Mar 10, 2023

Why not allow both by default? I mean, pick the one which is present at the current directory. If both exists, I think it should simply abort, because then ls-lint can't tell which one is actually intended by the user. I'm afraid of that linting is accidentally passing or failing by this kind of implicit preference.

@loeffel-io
Copy link
Owner

With v2.x this is provided by the config option: https://github.com/loeffel-io/ls-lint/releases/tag/v2.0.0-beta.0

Feel free to try it out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants