Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor config file parsing #28

Merged
merged 3 commits into from
Nov 9, 2015
Merged

Refactor config file parsing #28

merged 3 commits into from
Nov 9, 2015

Commits on Aug 6, 2015

  1. Refactor config file parsing 

    The parse_config_file and convert_parsed_args_to_config_file_contents
methods were moved to a generic ConfigFileParser class, which can be
overridden in the ArgumentParser constructor using the
config_file_parser option.

Addresses bw2#21
    @lahwaacz
    lahwaacz committed Aug 6, 2015
    Configuration menu
    Copy the full SHA
    556fa5b View commit details
    Browse the repository at this point in the history

  2. Let the config file parser set the syntax description 

    The very convoluted code for showing examples was greatly simplified,
there is little benefit in showing true args instead of pseudo-args. For
advanced parsers, the full syntax can't be expressed in a simple
description, so we should also refer to the "upstream" documentation for
completeness.
    @lahwaacz
    lahwaacz committed Aug 6, 2015
    Configuration menu
    Copy the full SHA
    6e59417 View commit details
    Browse the repository at this point in the history

  3. Fix failing tests for formatting help description

    @lahwaacz
    lahwaacz committed Aug 6, 2015
    Configuration menu
    Copy the full SHA
    5931ee1 View commit details
    Browse the repository at this point in the history