Closed homm closed 10 years ago
I think current module layout is too redundant.
from CommonMark import CommonMark parser = CommonMark.DocParser() renderer = CommonMark.HTMLRenderer()
I believe it should be more like this:
from CommonMark import DocParser, HTMLRenderer parser = DocParser() renderer = HTMLRenderer()
or
import CommonMark parser = CommonMark.DocParser() renderer = CommonMark.HTMLRenderer()
This can be achieved in several ways:
CommonMark.py
CommonMark/__init__.py
I prefer second option.
Also as there are many symbols in CommonMark.py, it is better to add __all__ list.
__all__
Dones, thanks for the advice!
I think current module layout is too redundant.
I believe it should be more like this:
or
This can be achieved in several ways:
CommonMark.py
module without package (setuptools allows that)CommonMark/__init__.py
all important symbols.CommonMark/__init__.py
I prefer second option.
Also as there are many symbols in
CommonMark.py
, it is better to add__all__
list.