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

reviewdog: upgrade from grep based to fnmatch based, add per-repo set… #549

Merged
merged 3 commits into from
Mar 25, 2024

Conversation

thypon
Copy link
Member

@thypon thypon commented Mar 19, 2024

…ting

@thypon thypon force-pushed the features/per-repo-blocklist branch 4 times, most recently from 55a8efd to ff7830c Compare March 19, 2024 17:55
@thypon thypon force-pushed the features/per-repo-blocklist branch 5 times, most recently from acf35cc to d14e3ba Compare March 20, 2024 16:38
@thypon thypon requested a review from evq March 20, 2024 16:58
@thypon
Copy link
Member Author

thypon commented Mar 20, 2024

I'm not too fond of this change, but it will allow every repository to have its own security-action-blocklist.txt using an fnmatch compatible style blocklist for unuseful rules.

This might improve the situation for fargate situation.

@evq @tarikdem @hspencer77 WDYT?

@thypon thypon requested a review from a team as a code owner March 22, 2024 15:33
@thypon thypon force-pushed the features/per-repo-blocklist branch from 5f959c7 to 7207e35 Compare March 22, 2024 15:37
@thypon thypon merged commit c187936 into main Mar 25, 2024
10 checks passed
@thypon thypon deleted the features/per-repo-blocklist branch March 25, 2024 14:09
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.

1 participant