Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update minimal click version to allow case_sensitive option #339

Closed
wants to merge 1 commit into from

Conversation

rom1gal
Copy link

@rom1gal rom1gal commented Oct 12, 2020

When using the CLI within specific environment (in my case an Azure DevOps agent running Ubuntu 18.06) some librairies need to be upgraded for the CLI to work properly (click when running the 6.7 version triggers the stacktrace displayed here : #337)

@andrewmchen
Copy link
Contributor

Thanks for reporting and thank you for the fix. I'm wondering if we can just remove this option instead. https://github.com/databricks/databricks-cli/blob/master/databricks_cli/secrets/cli.py#L54.

@sushi1998

@rom1gal
Copy link
Author

rom1gal commented Oct 25, 2020

Actually I do not understand the purpose of that option since it is currently not possible to create a Azure KV-backed secret scope :/

@pietern
Copy link
Contributor

pietern commented Jul 1, 2022

The case_sensitive argument was removed in #347.

However, in the mean time we merged #471 which depends on click >= 7.0 for a different reason.

@pietern pietern closed this Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants