Mercurial > public > mercurial-scm > hg-stable
diff contrib/import-checker.py @ 34055:bc2535238de2
import-checker: allow relative import a module being checked
This would make the checker more friendly for 3rd-party code. For example,
In remotefilelog/x.py, it may have:
from . import shallowutils
That could trigger "relative import of stdlib module" if
"remotefilelog" was installed in the system. If the module being checked
conflicts with the system module, it makes sense to not treat that module as
system module. This patch makes it so.
Differential Revision: https://phab.mercurial-scm.org/D552
| author | Jun Wu <quark@fb.com> |
|---|---|
| date | Mon, 28 Aug 2017 13:43:25 -0700 |
| parents | 8a8dd6e4a97a |
| children | 41401f502c83 |
line wrap: on
line diff
--- a/contrib/import-checker.py Thu Aug 31 11:28:59 2017 -0700 +++ b/contrib/import-checker.py Mon Aug 28 13:43:25 2017 -0700 @@ -411,7 +411,8 @@ assign the symbol to a module-level variable. In addition, these imports must be performed before other local imports. This rule only applies to import statements outside of any blocks. - * Relative imports from the standard library are not allowed. + * Relative imports from the standard library are not allowed, unless that + library is also a local module. * Certain modules must be aliased to alternate names to avoid aliasing and readability problems. See `requirealias`. """ @@ -493,7 +494,10 @@ # __future__ is special since it needs to come first and use # symbol import. if fullname != '__future__': - if not fullname or fullname in stdlib_modules: + if not fullname or ( + fullname in stdlib_modules + and fullname not in localmods + and fullname + '.__init__' not in localmods): yield msg('relative import of stdlib module') else: seenlocal = fullname