databricks / databricks-cli

(Legacy) Command Line Interface for Databricks
Other
386 stars 234 forks source link

Doesn't seem to work with minimal version of click (i.e., click 6.7)? #348

Open bgreenwell opened 4 years ago

bgreenwell commented 4 years ago

Was not able to get any of the commands working until I upgraded from click 6.7 to click 7.1.2; for example, with click 6.7 I was getting:

$ databricks --version
.
.
.
    type=click.Choice(['AZURE_KEYVAULT', 'DATABRICKS'], case_sensitive=True),
TypeError: __init__() got an unexpected keyword argument 'case_sensitive'

If this is the case, then perhaps consider updating the requirements section in the setup.py file?

bgreenwell commented 4 years ago

Looks like the case_sensitive argument was added in 7.0; per the change log @ https://github.com/pallets/click/blob/master/CHANGES.rst:

Add case_sensitive=False as an option to Choice. :issue:569