| 1 |
W dniu sob, 03.03.2018 o godzinie 12∶48 +0100, użytkownik Ulrich Mueller |
| 2 |
napisał: |
| 3 |
> > > > > > On Sat, 3 Mar 2018, Michał Górny wrote: |
| 4 |
> |
| 5 |
> |
| 6 |
> > parser.add_argument( |
| 7 |
> > '-d', '--include-dev-profiles', choices=('y', 'n'), metavar='<y|n>', |
| 8 |
> > - default='n', |
| 9 |
> > + default='y', |
| 10 |
> > help='include dev profiles in dependency checks') |
| 11 |
> |
| 12 |
> It seems counter-intuitive for a simple binary option to require an |
| 13 |
> argument. What is wrong with specifying -d to enable the option, and |
| 14 |
> simply not specifying it to disable? |
| 15 |
> |
| 16 |
|
| 17 |
What is wrong is that a number of developers have historically not |
| 18 |
specified the option and broke stuff. Plus, it's infinitely silly to |
| 19 |
require people to explicitly specify the option to enable required |
| 20 |
behavior. |
| 21 |
|
| 22 |
-- |
| 23 |
Best regards, |
| 24 |
Michał Górny |
Archives