Only allowlist noqa et al at the start of a comment (#1157)

This commit is contained in:
Charlie Marsh 2022-12-08 23:10:36 -05:00 committed by GitHub
parent db815a565f
commit a0a6327fae
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 2 deletions

View File

@ -5,9 +5,11 @@ a = 4
#foo(1, 2, 3) #foo(1, 2, 3)
def foo(x, y, z): def foo(x, y, z):
contentet = 1 # print('hello') content = 1 # print('hello')
print(x, y, z) print(x, y, z)
# This is a real comment. # This is a real comment.
#return True #return True
return False return False
#import os # noqa: ERA001

View File

@ -4,7 +4,7 @@ use regex::Regex;
static ALLOWLIST_REGEX: Lazy<Regex> = Lazy::new(|| { static ALLOWLIST_REGEX: Lazy<Regex> = Lazy::new(|| {
Regex::new( Regex::new(
r"(?i)pylint|pyright|noqa|nosec|type:\s*ignore|fmt:\s*(on|off)|isort:\s*(on|off|skip|skip_file|split|dont-add-imports(:\s*\[.*?])?)|TODO|FIXME|XXX" r"^(?i)(?:pylint|pyright|noqa|nosec|type:\s*ignore|fmt:\s*(on|off)|isort:\s*(on|off|skip|skip_file|split|dont-add-imports(:\s*\[.*?])?)|TODO|FIXME|XXX)"
).unwrap() ).unwrap()
}); });
static BRACKET_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"^[()\[\]{}\s]+$").unwrap()); static BRACKET_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"^[()\[\]{}\s]+$").unwrap());