2021-04-13 00:53:27 |
Colin Watson |
description |
Currently, the import fascist makes warnings like:
"""
You should not import IDistribution from canonical.launchpad.interfaces._schema_circular_imports:
lp.bugs.browser.bugtarget
"""
If it included the file name and line number of the offending import, it would be much easier to fix the warning, particularly if it used an Emacs-friendly format like:
"""
lib/lp/bugs/browser/bugtarget.py:44: from canonical.launchpad.interfaces._schema_circular_imports import IDistribution
"""
or even:
"""
lib/lp/bugs/browser/bugtarget.py:44:
from canonical.launchpad.interfaces._schema_circular_imports import IDistribution
IDistribution not exported from _schema_circular_imports
""" |
Currently, the import pedant makes warnings like:
"""
You should not import IDistribution from lp._schema_circular_imports:
lp.bugs.browser.bugtarget
"""
If it included the file name and line number of the offending import, it would be much easier to fix the warning, particularly if it used an Emacs-friendly format like:
"""
lib/lp/bugs/browser/bugtarget.py:44: from lp._schema_circular_imports import IDistribution
"""
or even:
"""
lib/lp/bugs/browser/bugtarget.py:44:
from lp._schema_circular_imports import IDistribution
IDistribution not exported from _schema_circular_imports
""" |
|