feat(checker): add checker for searching unncessary parentheses in func

sphinx-contrib · Aug 27, 2024 · 23f1f9b · 23f1f9b
1 parent a9559ba
commit 23f1f9b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/sphinxlint/checkers.py b/sphinxlint/checkers.py
@@ -513,3 +513,16 @@ def check_dangling_hyphen(file, lines, options):
         stripped_line = line.rstrip("\n")
         if _has_dangling_hyphen(stripped_line):
             yield lno + 1, "Line ends with dangling hyphen"
+
+
+@checker(".rst", ".po", rst_only=False)
+def check_unnecessary_func_parentheses(filename, lines, options):
+    """Check for unnecessary parentheses in :func: roles.
+
+    Bad:  :func:`test()`
+    Good: :func:`test`
+    """
+    for lno, line in enumerate(lines, start=1):
+        match = rst.FUNC_ROLE_WITH_UNNECESSARY_PARENTHESES.search(line)
+        if match:
+            yield lno, f"Unnecessary parentheses in {match.group(0)!r}"
diff --git a/sphinxlint/rst.py b/sphinxlint/rst.py
@@ -280,3 +280,5 @@ def inline_markup_gen(start_string, end_string, extra_allowed_before=""):
 )
 
 ROLE_MISSING_CLOSING_BACKTICK_RE = re.compile(rf"({ROLE_HEAD}`[^`]+?)[^`]*$")
+
+FUNC_ROLE_WITH_UNNECESSARY_PARENTHESES = re.compile(r"(^|\s):func:`[^`]+\(\)`")
Original file line number	Diff line number	Diff line change
Expand Up		@@ -280,3 +280,5 @@ def inline_markup_gen(start_string, end_string, extra_allowed_before=""):
		)

		ROLE_MISSING_CLOSING_BACKTICK_RE = re.compile(rf"({ROLE_HEAD}`[^`]+?)[^`]*$")

		FUNC_ROLE_WITH_UNNECESSARY_PARENTHESES = re.compile(r"(^\|\s):func:`[^`]+\(\)`")