-
-
Notifications
You must be signed in to change notification settings - Fork 776
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
Create new issue template for resolution of CodeQL alert #5242
Comments
This comment was marked as outdated.
This comment was marked as outdated.
Readable version of template above - use for reviewingPrerequisite
OverviewAs developers. we need to analyze CodeQL query alert INSERT-ALERTID and to either recommend dismissal of the alert or update the code to resolve the alert. Action Items
For merge team/dev lead
Resources/Instructions
|
When #5059 is merged, this issue may be unnecessary, because the codeql.yml workflow will automatically generate issues to address open CodeQL alerts from the codebase. The only possible reason we would require an issue template for resolution of CodeQL alerts is if a CodeQL alert appears in a Pull Request and the assignee cannot resolve it, and we want a resolution prior to merging the code. |
Dependency
Overview
We require a generic issue template that can be used to resolve all CodeQL alerts. The only information required to target a particular alert is the alert number, which provides a link to the alert detail page, from which all details such as file path are available, along with an explanation of the alert and recommendations for resolution.
Action Items
.github/ISSUE_TEMPLATE/resolve-CodeQL-alert.md
Content of issue template
Resources/Instructions
Testing Issue Templates
The text was updated successfully, but these errors were encountered: