-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add Hadolint #4768
Comments
I can work on this, is this still relevant? The problem is that still pre-commit throws some errors on some files that need to be fixed or in some cases ignored. However they could be solved in a different issue. I can work on this also fix them so that the pre-commit comes clean right of the box. |
Feel free to use this pre-commit as a starting point:
It does seem like it's not maintained; finding an alternative may be beneficial. |
Description
Adding hadolint to pre-commit
Rationale
Linting Dockerfile. Current lint warnings:
The text was updated successfully, but these errors were encountered: