Merge pull request #18 from MichaelKim0407/develop

release 1.3

README.md

@@ -62,6 +62,11 @@ or set `percent-greedy=<level>` and `format-greedy=<level>` in the `.flake8` con
 
 This plugin can also check for strings that appear to be f-strings but don't have the `f` prefix.
 
+`FS003` will be reported if a string:
+* is not prefixed with `f`
+* contains at least one pair of `{}`
+* is not prefixed with `r` (because r-strings are often regexes that would contain `{}`)
+
 Due to the potential for false positives, this check (`FS003`) is disabled by default.
 To enable this check,
 add the `--enable-extensions=FS003` command line option,

flake8_use_fstring/__init__.py

@@ -1 +1 @@
-__version__ = '1.2.1'
+__version__ = '1.3'

flake8_use_fstring/prefix.py

@@ -13,7 +13,8 @@
 
 FSTRING_REGEX = _re.compile(r'^([a-zA-Z]*?[fF][a-zA-Z]*?){1}["\']')
 NON_FSTRING_REGEX = _re.compile(
-    r'^[a-zA-Z]*(?:\'\'\'|\'|"""|")(.*?{.+?}.*)(?:\'|\'\'\'|"|""")$',
+    r'^[a-zA-Z]*(?<!\b[rR])(?:\'\'\'|\'|"""|")'
+    r'(.*?{.+?}.*)(?:\'|\'\'\'|"|""")$',
 )
 
 

tests/example.py

@@ -47,7 +47,8 @@ def format(self):  # noqa: A003
 
 # match missing prefix
 o = ('{n}'
-     '{{m}} {n}')
+     '{{m}} {n}'
+     r'[a-z]{1,3}')  # Should not be matched.
 
 # no errors below; coverage
 ''.strip()