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

Allow rules to specify specific repositories to ignore or run in #726

Closed
wants to merge 1 commit into from

Conversation

kdenhartog
Copy link
Member

This is useful in the case where we want to make specific modifications to rules for only a select number of repositories. One example of this is that we want to allow expect usage, but still alert on unwrap in Search specific repositories. However, within Brave-core we want to make sure we alert on both. This should allow us to fork rules to make them specific to the repository they run on.

This is useful in the case where we want to make specific modifications
to rules for only a select number of repositories. One example of this
is that we want to allow expect usage, but still alert on unwrap in
Search specific repositories. However, within Brave-core we want to
make sure we alert on both. This should allow us to fork rules to
make them specific to the repository they run on.
@kdenhartog kdenhartog requested a review from thypon as a code owner December 17, 2024 23:54
@kdenhartog
Copy link
Member Author

Ideally we don't want to go this route of per-repo specific rules. I'll instead take an approach in line with the blocklists we're already using instead.

@kdenhartog kdenhartog closed this Dec 18, 2024
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