yostudios / Spritemapper

CSS Spritemap Generator

Home Page:http://yostudios.github.com/Spritemapper/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

I suggest to don't use relative import in SpriteMapper project

harobed opened this issue · comments

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