[PATCH 7 of 8] check-code: check comment for '.. note::' without two newlines

Simon Heimberg simohe at besonet.ch
Tue Nov 5 02:06:38 CST 2013


# HG changeset patch
# User Simon Heimberg <simohe at besonet.ch>
# Date 1383638431 -3600
#      Tue Nov 05 09:00:31 2013 +0100
# Branch stable
# Node ID 0d56501f3381b0066453cb238ada262bad114b3a
# Parent  c9c7cfa3440b22eb72cabcbcf3bde87bf8f905ab
check-code: check comment for '.. note::' without two newlines

Because string entries are replaced before matching, we must search for
the transformed pattern. But it seems to be quite unique and does not return
false matches. If it will, they can be listed as 3rd arg in pypats.

diff -r c9c7cfa3440b -r 0d56501f3381 contrib/check-code.py
--- a/contrib/check-code.py	Tue Nov 05 08:59:55 2013 +0100
+++ b/contrib/check-code.py	Tue Nov 05 09:00:31 2013 +0100
@@ -263,6 +263,7 @@
   ],
   # warnings
   [
+    (r'(^| )oo +xxxxoo[ \n][^\n]', "add two newlines after '.. note::'"),
   ]
 ]
 
diff -r c9c7cfa3440b -r 0d56501f3381 tests/test-check-code.t
--- a/tests/test-check-code.t	Tue Nov 05 08:59:55 2013 +0100
+++ b/tests/test-check-code.t	Tue Nov 05 09:00:31 2013 +0100
@@ -204,3 +204,42 @@
    don't use old-style two-argument raise, use Exception(message)
   [1]
 
+  $ cat > rst.py <<EOF
+  > """problematic rst text
+  > 
+  > .. note::
+  >     wrong
+  > """
+  > 
+  > '''
+  > 
+  > .. note::
+  > 
+  >     valid
+  > 
+  > new text
+  > 
+  >     .. note::
+  > 
+  >         also valid
+  > '''
+  > 
+  > """mixed
+  > 
+  > .. note::
+  > 
+  >   good
+  > 
+  >     .. note::
+  >         plus bad
+  > """
+  > EOF
+  $ $check_code -w rst.py
+  rst.py:3:
+   > .. note::
+   warning: add two newlines after '.. note::'
+  rst.py:26:
+   >     .. note::
+   warning: add two newlines after '.. note::'
+  [1]
+


More information about the Mercurial-devel mailing list