Skip to content

Add suppression comments #1845

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

Merged
merged 1 commit into from
Oct 11, 2023
Merged

Add suppression comments #1845

merged 1 commit into from
Oct 11, 2023

Conversation

rzhao271
Copy link
Contributor

Closes a false positive issue.

@rzhao271 rzhao271 enabled auto-merge (squash) October 11, 2023 16:36
@rzhao271 rzhao271 self-assigned this Oct 11, 2023
@rzhao271 rzhao271 requested a review from connor4312 October 11, 2023 16:37
@rzhao271 rzhao271 added this to the October 2023 milestone Oct 11, 2023
@connor4312
Copy link
Member

Every one of the recent codeql issues are false positives, or test code... will need to go through and add surpressions for the rest of them still

@rzhao271 rzhao271 merged commit a6543cb into main Oct 11, 2023
@rzhao271 rzhao271 deleted the rzhao271/table-sup branch October 11, 2023 19:03
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