# HG changeset patch
# User Jun Wu <quark@fb.com>
# Date 1496256114 25200
# Node ID 071423d0a584b6ad99f3d033b0acb56c34240d9f
# Parent  5d44d7d4076e5a96001b0f88c730fa7ea24a9e02
check-code: suggest policy.importmod

This forbids patterns like `from mercurial.cext import parsers` which breaks
pure.

diff -r 5d44d7d4076e -r 071423d0a584 contrib/check-code.py
--- a/contrib/check-code.py	Fri May 26 03:41:10 2017 +0200
+++ b/contrib/check-code.py	Wed May 31 11:41:54 2017 -0700
@@ -340,6 +340,8 @@
     (r'^import pickle', "don't use pickle, use util.pickle"),
     (r'^import httplib', "don't use httplib, use util.httplib"),
     (r'^import BaseHTTPServer', "use util.httpserver instead"),
+    (r'^(from|import) mercurial\.(cext|pure|cffi)',
+     "use mercurial.policy.importmod instead"),
     (r'\.next\(\)', "don't use .next(), use next(...)"),
     (r'([a-z]*).revision\(\1\.node\(',
      "don't convert rev to node before passing to revision(nodeorrev)"),