Merge pull request #84 from coreruleset/chain_rule_comments

Add chain rule commenting guidance
coreruleset · Apr 17, 2023 · 7207c2d · 7207c2d
2 parents 76c7eed + 2af142e
commit 7207c2d
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/content/development/contribution_guidelines.md b/content/development/contribution_guidelines.md
@@ -37,8 +37,9 @@ chapter: false
 * Files must end with a single newline character.
 * No trailing whitespace at EOL.
 * No trailing blank lines at EOF (only the required single EOF newline character is allowed).
-* Add comments where possible and clearly explain any new rules.
 * Adhere to an 80 character line length limit where possible.
+* Add comments where possible and clearly explain any new rules.
+* Comments must not appear between chained rules and should instead be placed before the start of a rule chain.
 * All [chained rules](https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual(v2.x)#chain) should be indented like so, for readability:
 ```
 SecRule .. .. \