I suggest to don't use relative import in SpriteMapper project
harobed opened this issue · comments
Stéphane Klein commented
Hi,
I suggest to don't use this syntax :
from . import SpriteRef
from .config import CSSConfig
from .css import split_declaration
but use this syntax :
from spritemapper import SpriteRef
from spritemapper.config import CSSConfig
from spritemapper.css import split_declaration
Generally, I don't see and I don't use relative import.
Usual python project convention don't use relative import.
Lately, I've executed this command :
$ python ../Spritemapper/spritecss/finder.py mime.css
Traceback (most recent call last):
File "../Spritemapper/spritecss/finder.py", line 7, in <module>
from . import SpriteRef
ValueError: Attempted relative import in non-package
If the project don't use relative import, I would not get this error.
What do you think about this suggestion ?
I can fix it if you want ?
Regards,
Stéphane