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.

Sign up for GitHub

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

Jump to bottom

ci(semgrep): Update .semgrepignore #547

Merged
merged 1 commit into from
Oct 31, 2023
Merged

ci(semgrep): Update .semgrepignore #547

merged 1 commit into from
Oct 31, 2023

Conversation

evansims
Copy link
Member

Changes

This PR updates the .semgrepignore ignore file for our Semgrep security tooling. It adds the auth0/test and auth0/test_async directories to the ignore list so our tooling stops raising false positives by identifying unit test files as production code.

References

N/A

Testing

N/A

Checklist

@evansims
ci(semgrep): Update .semgrepignore
96e50f0 
Signed-off-by: Evan Sims <hello@evansims.com>
@evansims evansims temporarily deployed to internal October 31, 2023 16:48 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 31, 2023 16:48 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 31, 2023 16:48 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal October 31, 2023 16:57 — with GitHub Actions Inactive
@evansims evansims marked this pull request as ready for review October 31, 2023 16:58
@evansims evansims requested a review from a team as a code owner October 31, 2023 16:58
@evansims evansims merged commit d8b42ae into master Oct 31, 2023
15 checks passed
@evansims evansims deleted the ci/semgrep/update-semgrepignore branch October 31, 2023 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Widcket Widcket Widcket approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@evansims @Widcket