-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
chore: fix hadolint/shellcheck violations #320
Conversation
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
… tag (tracked by dependabot) Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
7bb0a3f
to
120dee9
Compare
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
…l chars Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
14f8985
to
c06df26
Compare
Possible cause of JENKINS-70157? Of all the commits I looked through between 3071.v7e9b_0dc08466-1 and 3071.v7e9b_0dc08466-5 this seemed like the most likely candidate. CC @dduportal |
That could also be impacting ci.jenkins.io. Currently checking, thanks for the notice @basil |
I can't reproduce the issue on my Windows setup any help is welcome. |
This PR fixes a set of hadolint/shellcheck violations on the images.
Tried to proceed 1 commit per rule, so we can easily rollback only one.
I'm not sure yet how to efficiently load hadolint recursively on ALL
Dockerfile
though.