taurus-org / taurus

Moved to https://gitlab.com/taurus-org/taurus
http://taurus-scada.org
43 stars 46 forks source link

Allow any case in CLI --log option #1112

Closed cpascual closed 4 years ago

cpascual commented 4 years ago

The l--taurus-log-level option of old CLI scripts was replaced by the --log option to the taurus script (#856). The old one was case-insensitive, while the new one is not.. Make the new one insensitive. For example, allow :

# only this is valid before this PR
taurus --log-level=Debug form

# this should be also valid now
taurus --log-level=debug form
cpascual commented 4 years ago

@taurus-org/integrators , please have a look at this trivial PR

cpascual commented 4 years ago

This kwarg is not accepted by click 6.6 (Debian strech). Do we want to raise the click requirement?

Ups... that's a very good point! No, I would not raise the requirement above the stretch defaults just for this. I'll propose a conditional implementation