[Python-Dev] ConfigParser argparse integration module (original) (raw)

Łukasz Langa lukasz at langa.pl
Sat Oct 13 10:06:55 CEST 2012

Wiadomość napisana przez S Joshua Swamidass <swamidass at gmail.com> w dniu 13 paź 2012, o godz. 05:24:

Hello,

It seemed like there were several requests to enable ini file parsing with argparse. I wrote module to do this. I'd be curious to know if this is useful and anyone has a better approach.

This mailing list focuses on the development of Python so your message if off-topic for this list. I suggest python-announce-list at python.org or the generic python-list at python.org.

http://pypi.python.org/pypi/ConfArgParse

That being said, a couple of remarks:

  1. See configglue which does what your module tries to do.
  2. A "Production/Stable" trove classifier without any unit tests does not look very convincing.
  3. Your code is 2.x only.
  4. ConfArgParse.args2config uses RawConfigParser whereas ConfArgParse._parse_config uses SafeConfigParser which will lead to inconsistent content handling.
  5. Your code is not PEP8 compliant.
  6. Long option names traditionally don't use underscores but dashes. E.g. not --conf_file but --conf-file.
  7. There doesn't seem to be a homepage. The one you listed in setup.py is your homepage, not the project's homepage.
  8. There doesn't seem to be a code repository for the project.

All in all: nice approach, promising project but definitely needs more work. Good luck :)

-- Best regards, Łukasz Langa Senior Systems Architecture Engineer

IT Infrastructure Department Grupa Allegro Sp. z o.o.

http://lukasz.langa.pl/ +48 791 080 144

-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.python.org/pipermail/python-dev/attachments/20121013/eba071c8/attachment.html>

More information about the Python-Dev mailing list