Skip to content
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

2361 - Added hadolint config and set failure threshold to 'error'. #529

Merged
merged 5 commits into from Sep 1, 2023
Merged

Conversation

ghost
Copy link

@ghost ghost commented Aug 2, 2023

Added hadolint config, set failure threshold to "error", and added a check/run that checks out the forked repository head when triggered by a pull request to prevent the workflow trying to run in a detached head state (and thereby signalling a false pass).

@ghost ghost added draft CI/CD Changes to the CI/CD Pipeline labels Aug 2, 2023
@ghost ghost removed the draft label Aug 2, 2023
@ghost
Copy link
Author

ghost commented Aug 2, 2023

Super-Linter and actions/checkout do not natively handle pull requests from forks well so I added a check for whether or not the workflow was triggered by a pull request and, if so, ensures it first checks out the PR head and sets it locally to track the origin/master branch so when Super-Linter does it's built-in pull request everything lines up.

@cfkoehler cfkoehler self-requested a review August 17, 2023 09:36
@cfkoehler cfkoehler added this to the v8.0.0-M3 milestone Aug 17, 2023
contrib/docker/.hadolint.yaml Show resolved Hide resolved
.github/workflows/super-linter.yaml Outdated Show resolved Hide resolved
.github/workflows/super-linter.yaml Outdated Show resolved Hide resolved
@jpdahlke jpdahlke modified the milestones: v8.0.0-M3, v8.0.0-M4 Aug 30, 2023
@cfkoehler cfkoehler merged commit 0c7073d into NationalSecurityAgency:master Sep 1, 2023
8 checks passed
@ghost ghost deleted the 2361-lint-fix branch September 6, 2023 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Changes to the CI/CD Pipeline
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants