Python: Broaden noqa regex

yoff · yoff · commit 98301332bd57 · 2022-06-16T11:16:58.000+02:00
diff --git a/python/ql/src/analysis/AlertSuppression.ql b/python/ql/src/analysis/AlertSuppression.ql
@@ -72,7 +72,7 @@ class LgtmSuppressionComment extends LineSuppressionComment {
  * A noqa suppression comment. Both pylint and pyflakes respect this, so lgtm ought to too.
  */
 class NoqaSuppressionComment extends LineSuppressionComment {
-  NoqaSuppressionComment() { this.getContents().toLowerCase().regexpMatch("\\s*noqa\\s*") }
+  NoqaSuppressionComment() { this.getContents().toLowerCase().regexpMatch("\\s*noqa([^:].*)?") }
 
   override string getAnnotation() { result = "lgtm" }
 }

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ class LgtmSuppressionComment extends LineSuppressionComment {`
`72`	`72`	`* A noqa suppression comment. Both pylint and pyflakes respect this, so lgtm ought to too.`
`73`	`73`	`*/`
`74`	`74`	`class NoqaSuppressionComment extends LineSuppressionComment {`
`75`		`- NoqaSuppressionComment() { this.getContents().toLowerCase().regexpMatch("\\snoqa\\s") }`
	`75`	`+ NoqaSuppressionComment() { this.getContents().toLowerCase().regexpMatch("\\snoqa([^:].)?") }`
`76`	`76`
`77`	`77`	`override string getAnnotation() { result = "lgtm" }`
`78`	`78`	`}`