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

Add Regal linting in CI pipeline #515

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

anderseknert
Copy link
Contributor

Hello kubescape friends!

Regal integration in CI just made simpler thanks to the new setup-regal GitHub Action, so I figured I'd check back :)

The new-ish --format github flag will also annotate PRs with any violation encountered in the code at the location of the violation, making it really easy to spot.

Also fix some style violations from more recent Regal rules, like excessive whitespace or comments without whitespace. That made the PR a little noisy, but it should be fairly quick to skim through.

Let me know what you all think, and have a great weekend!

Simplified now by the new [setup-regal](https://github.com/StyraInc/setup-regal) GitHub Action.
The new-ish `--format github` flag will also annotate PRs with any violation encountered
in the code at the location of the violation, making it really easy to spot.

Also fixed some style violations from more recent Regal rules, and amended the Regal
configuration to ignore some of the new rules that would require more work to enable.

Let me know what you all think, and have a great weekend!

Signed-off-by: Anders Eknert <[email protected]>
@alegrey91
Copy link
Contributor

Hi @anderseknert, welcome back!
Thanks for your contribution, this is really appreciated.
I pushed your changes into a branch of mine to run the tests that make use of internal organization secrets.
PR here: #516
Once all the tests passed, we can merge your PR! ;)

@alegrey91 alegrey91 self-requested a review October 6, 2023 16:26
Copy link
Contributor

@alegrey91 alegrey91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alegrey91 alegrey91 merged commit a94b1f2 into kubescape:master Oct 6, 2023
32 of 34 checks passed
@anderseknert anderseknert deleted the regal-cicd branch October 6, 2023 16:53
@anderseknert
Copy link
Contributor Author

Fantastic! Thanks @alegrey91 👍 Would love to hear about your experience with Regal later. Until then, take care!

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