I really have to find a vim script to format PEP8 compliant a python code (I know, you use emacs and don't care ;))
Thanks for this part:
647 - if __python_name_data_directory__.startswith('/'):
648 - pathname = __python_name_data_directory__
649 - else:
650 - pathname = os.path.dirname(__file__) + '/' + __python_name_data_directory__
651 + # Get pathname absolute or relative.
652 + path = os.path.join(
653 + os.path.dirname(__file__), __python_name_data_directory__)
I didn't know that joining two absolute path only take the second. Handy trick :)
Just a question, what is the __all__ in python_nameconfig.py is for? I only know this for autoimporting module in __init__.py, and you seem to still import function one by one:
from python_name.python_nameconfig import get_data_file
So, I don't really see its usefulness ;)
also, why importing multiple module in a same line is wrong?
(cf commit 404)
BTW, merged, thanks! I'll try to go file by file to include PEP8 compliance, but this is a long term work :)
Hey Jonathan, thanks for your path.
I really have to find a vim script to format PEP8 compliant a python code (I know, you use emacs and don't care ;))
Thanks for this part: name_data_ directory_ _.startswith( '/'): name_data_ directory_ _ dirname( __file_ _) + '/' + __python_ name_data_ directory_ _ dirname( __file_ _), __python_ name_data_ directory_ _)
647 - if __python_
648 - pathname = __python_
649 - else:
650 - pathname = os.path.
651 + # Get pathname absolute or relative.
652 + path = os.path.join(
653 + os.path.
I didn't know that joining two absolute path only take the second. Handy trick :)
Just a question, what is the __all__ in python_ nameconfig. py is for? I only know this for autoimporting module in __init__.py, and you seem to still import function one by one: name.python_ nameconfig import get_data_file
from python_
So, I don't really see its usefulness ;)
also, why importing multiple module in a same line is wrong?
(cf commit 404)
BTW, merged, thanks! I'll try to go file by file to include PEP8 compliance, but this is a long term work :)