search

loganasherjones / yapconf

Yet Another Python Configuration
http://yapconf.readthedocs.io/en/stable/
MIT License
18 stars 4 forks source link

Made argparse items never required. Fixes #42 #62

Closed loganasherjones closed 6 years ago

loganasherjones commented 6 years ago

Argparse arguments should never really be marked as required. We don't have enough information to mark them as required. Everything yapconf does with the CLI is just helpful options. We assume that if they are not provided on the command-line, they will be added by a different source later.

codecov-io commented 6 years ago

Codecov Report

Merging #62 into master will decrease coverage by 0.05%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   94.54%   94.49%   -0.06%     
==========================================
  Files           4        4              
  Lines         605      599       -6     
  Branches      143      141       -2     
==========================================
- Hits          572      566       -6     
  Misses         21       21              
  Partials       12       12
Impacted Files Coverage Δ
yapconf/items.py 94.47% <ø> (-0.08%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 220d961...c7d7679. Read the comment docs.