You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@yetus.apache.org by GitBox <gi...@apache.org> on 2022/09/01 17:34:19 UTC
[GitHub] [yetus] GauthamBanasandra opened a new pull request, #287: YETUS-1199. Add ability to ignore hadolint warnings
GauthamBanasandra opened a new pull request, #287:
URL: https://github.com/apache/yetus/pull/287
* This PR adds the option
--hadolint-ignore-list which
can be used to ignore the
warnings detected by
hadolint.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@yetus.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [yetus] GauthamBanasandra commented on pull request #287: YETUS-1199. Add ability to ignore hadolint warnings
Posted by GitBox <gi...@apache.org>.
GauthamBanasandra commented on PR #287:
URL: https://github.com/apache/yetus/pull/287#issuecomment-1247580905
> Using .hadolint.yaml file
I'll make peace with this. Closing this PR. Thanks @aw-was-here .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@yetus.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [yetus] GauthamBanasandra closed pull request #287: YETUS-1199. Add ability to ignore hadolint warnings
Posted by GitBox <gi...@apache.org>.
GauthamBanasandra closed pull request #287: YETUS-1199. Add ability to ignore hadolint warnings
URL: https://github.com/apache/yetus/pull/287
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@yetus.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [yetus] aw-was-here commented on pull request #287: YETUS-1199. Add ability to ignore hadolint warnings
Posted by GitBox <gi...@apache.org>.
aw-was-here commented on PR #287:
URL: https://github.com/apache/yetus/pull/287#issuecomment-1234673160
This ability already exists in two ways, both native to the hadolint binary without needing a code change in Apache Yetus:
* Using [hadolint ignore directives](https://github.com/apache/yetus/blob/main/precommit/src/main/shell/test-patch-docker/Dockerfile#L43)
* Using .hadolint.yaml file
Both documented on the [hadolint page](https://github.com/hadolint/hadolint#configure)
Since precommit doesn't really do this extra work for other tools without a good reason, I'd like to hear what the reasons are these other methods don't work. Otherwise, I'm very much a -1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@yetus.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org