Skip to content
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

QL4QL: Add query suggesting use of inline test expectations #18767

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

hvitved
Copy link
Contributor

@hvitved hvitved commented Feb 13, 2025

No description provided.

Comment on lines +9 to +11
/**
* Represents a line comment.
*/

Check warning

Code scanning / CodeQL

Class QLDoc style. Warning test

The QLDoc for a class should start with 'A', 'An', or 'The'.
@hvitved hvitved added the no-change-note-required This PR does not need a change note label Feb 13, 2025
@hvitved hvitved changed the title QL4QL: Add query suggestion use of inline test expectations QL4QL: Add query suggesting use of inline test expectations Feb 13, 2025
@hvitved hvitved marked this pull request as ready for review February 13, 2025 10:28
@hvitved hvitved requested a review from a team as a code owner February 13, 2025 10:28
@hvitved hvitved force-pushed the ql4ql/inline-expectations branch from 5336cb7 to f930dcc Compare February 13, 2025 12:01
@hvitved hvitved force-pushed the ql4ql/inline-expectations branch from f930dcc to d14cbf8 Compare February 13, 2025 12:01
@@ -0,0 +1 @@
queries/style/QlRefInlineExpectations.ql

Check warning

Code scanning / CodeQL

Query test without inline test expectations Warning test

Query test does not use inline test expectations.
@@ -0,0 +1 @@
query: queries/style/OmittableExists.ql

Check warning

Code scanning / CodeQL

Query test without inline test expectations Warning test

Query test does not use inline test expectations.
@hvitved hvitved marked this pull request as draft February 13, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-change-note-required This PR does not need a change note QL-for-QL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant