[PATCH 3 of 8] docchecker: introduce a way to check for poor markup

timeless timeless at mozdev.org
Sun Jan 10 13:36:17 CST 2016


# HG changeset patch
# User timeless <timeless at mozdev.org>
# Date 1452109634 0
#      Wed Jan 06 19:47:14 2016 +0000
# Node ID 80f0423c60c98bc177f506cba62c492ed98d257b
# Parent  ad3e5e1b9f4d3ec45462d244ea4757a69a954621
docchecker: introduce a way to check for poor markup

Specifically, :hg:`foo "bar baz"` when rendered by `hg help`
results in:

  "hg foo "bar baz""

... which is hard to read.

We encourage :hg:`foo 'bar baz'` instead.

diff --git a/doc/docchecker b/doc/docchecker
new file mode 100755
--- /dev/null
+++ b/doc/docchecker
@@ -0,0 +1,50 @@
+#!/usr/bin/env python
+#
+# docchecker - look for problematic markup
+#
+# Copyright 2016 timeless <timeless at mozdev.org> and others
+#
+# This software may be used and distributed according to the terms of the
+# GNU General Public License version 2 or any later version.
+import sys
+import re
+
+leadingline = re.compile(r'(^\s*)(\S.*)$')
+hg_backtick = re.compile(r':hg:`[^`]*"[^`]*`')
+
+def check(line):
+  if hg_backtick.search(line):
+    print(line)
+    print('warning: please avoid nesting " in `')
+
+def work(file):
+  (llead, lline) = ('', '')
+
+  for line in file:
+    # this section unwraps lines
+    match = leadingline.match(line)
+    if not match:
+      check(lline)
+      (llead, lline) = ('', '')
+      continue
+
+    lead, line = match.group(1), match.group(2)
+    if (lead == llead):
+      if (lline != ''):
+        lline += ' ' + line
+      else:
+        lline = line
+    else:
+      check(lline)
+      (llead, lline) = (lead, line)
+  check(lline)
+
+def main():
+  for f in sys.argv[1:]:
+    try:
+      with open(f) as file:
+        work(file)
+    except:
+      print("failed to process %s" % f)
+
+main()


More information about the Mercurial-devel mailing list