-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add code-quality-extended query suites #19808
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces “extended” code-quality query suites that select all queries tagged with quality
, and updates integration tests and expectations accordingly.
- Added
*-code-quality-extended.qls
suite files for each supported language, applying the new selectors and suite-helpers. - Updated each language’s integration-tests
well_known_query_suites
to include the extended suite. - Created or updated
.expected
files to list the queries now included in the extended suites and removed them fromnot_included_in_qls.expected
.
Reviewed Changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated no comments.
Show a summary per file
File Pattern | Description |
---|---|
/src/codeql-suites/-code-quality-extended.qls | New extended code-quality suite that applies the extended selectors |
/integration-tests//test.py | Include *-code-quality-extended.qls in the test suite |
/integration-tests//*-code-quality-extended.qls.expected | List of queries expected in the extended suite (may be empty) |
/integration-tests//not_included_in_qls.expected | Removed entries for queries now covered by the extended suite |
misc/suite-helpers/code-quality-extended-selectors.yml | New selector config to pick queries tagged quality |
Comments suppressed due to low confidence (3)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
PR needs to be rebased as #19783 has been merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JS Looks good to me! I have a feeling we’ll end up with quite a few pull requests that have conflicts. 😆
ca09b13
to
fcdc12b
Compare
fcdc12b
to
94052f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This PR is adding the code-quality-extended selector and query suite files. The content of the query suites largely match the code-quality suites, because of the not yet added
quality
tags. Rust and Java differences are shown in thenot_included_in_qls.expected
files.