Skip default role check for multiline roles in tables. (#65)

Author: ezio-melotti

sphinxlint/checkers.py

@@ -117,6 +117,10 @@ def check_default_role(file, lines, options=None):
         if match:
             before_match = line[: match.start()]
             after_match = line[match.end() :]
+            stripped_line = line.strip()
+            if (stripped_line.startswith("|") and stripped_line.endswith("|") and
+                stripped_line.count("|") >= 4 and "|" in match.group(0)):
+                return  # we don't handle tables yet.
             if re.search(rst.ROLE_TAG + "$", before_match):
                 # It's not a default role: it starts with a tag.
                 continue

tests/fixtures/xfail/default-role-in-tables.rst

@@ -0,0 +1,7 @@
+.. expect: default role used (hint: for inline literals, use double backticks) (default-role)
+
+In the following table there are a couple of default roles that should fail:
+
++-----------------+-------------------------------+----------------------------+
+| `French`        | Julien Palard `@JulienPalard` | `GitHub <github_fr_>`_     |
++-----------------+-------------------------------+----------------------------+

tests/fixtures/xpass/multiline-links-in-table.rst

@@ -0,0 +1,6 @@
+Links and default roles that span multiple lines in tables should be ignored:
+
++-----------------+-------------------------------+----------------------------+
+| `French (fr)    | Julien Palard (`@JulienPalard | `GitHub <github_fr_>`_     |
+| <doc_fr_>`_     | on GitHub`)                   |                            |
++-----------------+-------------------------------+----------------------------+