Skip to content

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 2 times, most recently from f930dcc to d14cbf8 Compare February 13, 2025 12:01
@@ -0,0 +1 @@
queries/style/QlRefInlineExpectations.ql No newline at end of file

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 No newline at end of file

Check warning

Code scanning / CodeQL

Query test without inline test expectations Warning test

Query test does not use inline test expectations.
@hvitved hvitved force-pushed the ql4ql/inline-expectations branch from dc41e88 to 9f9857b Compare February 13, 2025 15:26
@hvitved hvitved marked this pull request as ready for review February 13, 2025 19:09
@hvitved hvitved requested a review from asgerf February 17, 2025 08:57
@hvitved hvitved merged commit 3644de0 into github:main Feb 17, 2025
11 checks passed
@hvitved hvitved deleted the ql4ql/inline-expectations branch February 17, 2025 09:11
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.

2 participants