Rename the CCR query suite to code-quality#18974
Conversation
|
We need to make sure that is also updated. |
|
Would also need to update the known packs in spark: |
f78e726 to
97adaa9
Compare
@rvermeulen Thank you. Done in 97adaa9. |
I don't know anything about this. It looks like we already have a "quality" suite there, should that be renamed? Can we rename entries here without breaking other tools/queries? Should we keep the CCR name? |
There was a problem hiding this comment.
Pull Request Overview
This PR renames all references of the "CCR" query suite to "code-quality" to align file naming conventions.
- Update the "packs" list by replacing "ccr" with "code-quality".
- Update the warning message to reflect the new suite name "code-quality".
Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more
RasmusWL
left a comment
There was a problem hiding this comment.
You need to also rename the C++ suite: https://github.com/github/codeql/blob/main/cpp/ql/src/codeql-suites/cpp-ccr.qls
Changes made to misc/scripts/generate-code-scanning-query-list.py LGTM, but do get a green run of the Build code scanning query list job before merging 👍
97adaa9 to
a021492
Compare
a021492 to
6cd49f4
Compare
michaelnebel
left a comment
There was a problem hiding this comment.
C# LGTM
Is there a PR for updating the corresponding DCA source suite?
6cd49f4 to
b5684fd
Compare
This PR renames all
CCR.qlsfiles tocode-quality.qls.