You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This way a PR is not released for merge unless the linter is fine. An important prerequisite is that the introduction of this action also cleans all linter warnings that would lead to a PR being rejected. Otherwise contributors are forced to clean up code they didn't change.
Also a good documentation on how to invoke the linter locally would be helpful.
(I am not a fan of pre-commit hooks, as they break the commit-often paradigm, but I also don't want to wait for actions to finish just to get all linter warnings fixed.)
Different linters would be available for this project, e.g. for reuse or for ansible itself. Do we want to run those through GitHub actions?
The text was updated successfully, but these errors were encountered: