Skip to content

Commit cd5940a

Browse files
authored
Update change note requirements and reporting language
Clarified the requirement for change notes in pull requests and updated wording for reporting checks.
1 parent 38878e5 commit cd5940a

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

.github/copilot-instructions.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
11
A change note is required for any pull request which modifies:
2-
32
- The structure or layout of the release artifacts.
43
- The evaluation performance (memory, execution time) of an existing query.
54
- The results of an existing query in any circumstance.
6-
If only adding new rule queries, a change note is not required.
75

6+
If the pull request only adds new rule queries, a change note is not required.
87
Confirm that either a change note is not required or the change note is required and has been added.
98

109
For PRs that add new queries or modify existing queries, also consider the following review checklist:
11-
1210
- Confirm that the output format of shared queries is valid.
1311
- Have all the relevant rule package description files been checked in?
1412
- Have you verified that the metadata properties of each new query is set appropriately?
@@ -19,4 +17,4 @@ For PRs that add new queries or modify existing queries, also consider the follo
1917
- Does the query not reinvent features in the standard library?
2018
- Can the query be simplified further (not golfed!)
2119

22-
Only report checks that failed
20+
Only report checks that fail

0 commit comments

Comments
 (0)