<div class="snippet-clipboard-content notranslate position-relative overflow-auto" data-snippet-clip

hadolint_output is empty about hadolint-gh-action HOT 6 OPEN

jbergstroem commented on June 25, 2024

hadolint_output is empty

from hadolint-gh-action.

Comments (6)

jbergstroem commented on June 25, 2024 1

Thanks for the bug report. Will look at it tonight!

from hadolint-gh-action.

jbergstroem commented on June 25, 2024 1

Been chasing this for a bit. I believe that parsing %0A (newline escaping) might have changed in github actions.

from hadolint-gh-action.

jbergstroem commented on June 25, 2024 1

Any update on this?

Been busy with work; will make an re-attempt at the fix this week. Sorry for the delay.

from hadolint-gh-action.

jbergstroem commented on June 25, 2024

Was able to repro with the draft PR as a e2e test. Here's the problem:

++ eval hadolint --no-fail --no-color '' -f json test/fixtures/Dockerfile-warning
+++ hadolint --no-fail --no-color -f json test/fixtures/Dockerfile-warning
+ OUTPUT='[{"code":"DL3018","column":1,"file":"test/fixtures/Dockerfile-warning","level":"warning","line":3,"message":"Pin versions in apk add. Instead of `apk add <package>` use `apk add <package>=<version>`"}]'
+ echo 'hadolint_output=[{"code":"DL3018","column":1,"file":"test/fixtures/Dockerfile-warning","level":"warning","line":3,"message":"Pin versions in apk add. Instead of `apk add <package>` use `apk add <package>=<version>`"}] >> /dev/null'
hadolint_output=[{"code":"DL3018","column":1,"file":"test/fixtures/Dockerfile-warning","level":"warning","line":3,"message":"Pin versions in apk add. Instead of `apk add <package>` use `apk add <package>=<version>`"}] >> /dev/null

Note: the e2e test is incorrect as well.

from hadolint-gh-action.