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

ci: various improvements #122

Merged
merged 3 commits into from
Jun 7, 2022
Merged

ci: various improvements #122

merged 3 commits into from
Jun 7, 2022

Conversation

willfindlay
Copy link
Contributor

See individual commit messages for context.

  • Rename checkpatch's static-checks job
  • Add a job to lint our helm chart
  • Add a help message to the generated-files job

This name was left over from when I factored this job out of static checks. The name
"checkpatch" makes more sense, so let's rename it to avoid confusion with the other job.

Signed-off-by: William Findlay <[email protected]>
@willfindlay willfindlay force-pushed the pr/willfindlay/ci-improvements branch from 09c07ff to 69609cf Compare June 3, 2022 16:18
This job runs install/kubernetes/test.sh and verifies that docs are up to date.

Signed-off-by: William Findlay <[email protected]>
Add a message to the generated-files job that explains what to do to get the job passing.

Signed-off-by: William Findlay <[email protected]>
@willfindlay willfindlay force-pushed the pr/willfindlay/ci-improvements branch 2 times, most recently from e9f1415 to 796e977 Compare June 3, 2022 16:33
@willfindlay willfindlay marked this pull request as ready for review June 3, 2022 16:33
@willfindlay willfindlay requested review from a team, kevsecurity and jrfastab and removed request for a team and kevsecurity June 3, 2022 16:34
@jrfastab jrfastab merged commit 3c26162 into main Jun 7, 2022
@jrfastab jrfastab deleted the pr/willfindlay/ci-improvements branch June 7, 2022 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants