Skip to content

Remove excluded CodeQL rules #51401

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
Closed
jakebailey opened this issue Nov 3, 2022 · 0 comments · Fixed by #51878
Closed

Remove excluded CodeQL rules #51401

jakebailey opened this issue Nov 3, 2022 · 0 comments · Fixed by #51878
Assignees
@jakebailey
Labels
Fix Available A PR has been opened for this issue Infrastructure Issue relates to TypeScript team infrastructure
Milestone
TypeScript 5.0.0

Comments

@jakebailey
Copy link
Member

In #51387, I modified our CodeQL config to exclude certain problematic rules; see github/codeql#10937.

Once the fixes are released and available in https://github.com/github/codeql-action, we should remove the exclusions.
@jakebailey jakebailey added this to the TypeScript 5.0.0 milestone Nov 3, 2022
@jakebailey jakebailey self-assigned this Nov 3, 2022
@RyanCavanaugh RyanCavanaugh added the Infrastructure Issue relates to TypeScript team infrastructure label Nov 3, 2022
@jakebailey jakebailey mentioned this issue Nov 4, 2022
Merged
@jakebailey jakebailey mentioned this issue Dec 13, 2022
Merged
@typescript-bot typescript-bot added the Fix Available A PR has been opened for this issue label Dec 13, 2022
jakebailey added a commit that referenced this issue Dec 13, 2022
@jakebailey
Remove excluded CodeQL rules, fix reference to Gulpfile (#51878)
a3802c1 
Fixes #51401
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jakebailey jakebailey

Labels
Fix Available A PR has been opened for this issue Infrastructure Issue relates to TypeScript team infrastructure
Projects
None yet
Milestone
TypeScript 5.0.0
Development

Successfully merging a pull request may close this issue.

Remove excluded CodeQL rules, fix reference to Gulpfile jakebailey/TypeScript
3 participants
@jakebailey @RyanCavanaugh @typescript-bot