W dniu sob, 03.03.2018 o godzinie 12∶48 +0100, użytkownik Ulrich Mueller
napisał:
> > > > > > On Sat, 3 Mar 2018, Michał Górny wrote:
> 
>  
> >     parser.add_argument(
> >             '-d', '--include-dev-profiles', choices=('y', 'n'), 
> > metavar='<y|n>',
> > -           default='n',
> > +           default='y',
> >             help='include dev profiles in dependency checks')
> 
> It seems counter-intuitive for a simple binary option to require an
> argument. What is wrong with specifying -d to enable the option, and
> simply not specifying it to disable?
> 

What is wrong is that a number of developers have historically not
specified the option and broke stuff. Plus, it's infinitely silly to
require people to explicitly specify the option to enable required
behavior.

-- 
Best regards,
Michał Górny


Reply via email to