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

Default running against cdw #324

Open
gonzoinc opened this issue Jun 13, 2024 · 1 comment
Open

Default running against cdw #324

gonzoinc opened this issue Jun 13, 2024 · 1 comment
Labels
Type: Enhancement New feature or request

Comments

@gonzoinc
Copy link

Please update the salt-lint command to default run against the current working directory. It is very odd to need to build out the find statement to run the linter against all files in a salt repo, that should be default action if no args are present.

@gonzoinc gonzoinc added the Type: Enhancement New feature or request label Jun 13, 2024
@NoRePercussions
Copy link

I'm not sure if it is very odd -- a quick survey of whether linters and checkers I think of assume this by default:

alejandra: no
ansible-lint: yes
black: no
flake8: yes
ruff: yes
shellcheck: no

I agree that this would be nice, though because salt-lint defaults to looking for init.sls when called on a directory, I don't think it would be as pretty as just defaulting to ..

Globbing is perhaps a briefer way to search in most modern shells (including bash with the globstar option): salt-lint **/*.sls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants