Skip to content

Commit 1cca8c3

Browse files
authored
Update autofix-instructions.md
1 parent fb7bd2e commit 1cca8c3

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

docs/autofix-instructions.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,11 @@ The full list of supported rules per standard is published as
4343
eliminates the alert. Do not refactor surrounding code, rename symbols,
4444
reformat unrelated lines, or change public APIs unless strictly required to
4545
satisfy the rule.
46-
- **No drive-by changes.** Do not add features, fix unrelated warnings, change
47-
build flags, update dependencies, or “improve” code that the alert does not
48-
point at.
46+
- **Multiple alerts.** Check if there are other code scanning alerts
47+
for the same coding standard reported for the same code location, fix all
48+
the relevant alerts at once in the same PR.
49+
- **No drive-by changes.** Do not add features, change build flags, update
50+
dependencies, or “improve” code that the alert does not point at.
4951
- **Do not attempt at fixing design issues.** A fix should not attempt to
5052
“improve” the design of the code or address architectural issues.
5153
Always verify that the code section around the alert is intended to follow
@@ -55,8 +57,6 @@ The full list of supported rules per standard is published as
5557
that a deviation should be added instead of a code fix.
5658
- **New code must comply with the same standard.** Any code modified by the
5759
fix must itself satisfy every rule of the coding standard being verified.
58-
Check if there are other code scanning alerts open for the same code location,
59-
try to fix all the relevant issues at once in the same PR.
6060
Cross-check the changed code against the COMPLIANT examples in the
6161
corresponding `test/rules/<rule-id>/` directory and against every other
6262
relevant rules (e.g. don’t fix an integer-conversion rule by introducing a

0 commit comments

Comments
 (0)