Skip to content

azure: use CLI config#4137

Merged
efiop merged 1 commit into
treeverse:masterfrom
efiop:fix-4129
Jun 30, 2020
Merged

azure: use CLI config#4137
efiop merged 1 commit into
treeverse:masterfrom
efiop:fix-4129

Conversation

@efiop
Copy link
Copy Markdown
Contributor

@efiop efiop commented Jun 30, 2020

Fixes #4129

  • ❗ I have followed the Contributing to DVC checklist.

  • 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Comment thread dvc/remote/azure.py Outdated
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worried about this dependency being too heavy, might go back to plain knack solution instead.

Comment thread dvc/remote/azure.py Outdated
@efiop efiop changed the title azure: use CLI config [WIP] azure: use CLI config Jun 30, 2020
@efiop efiop force-pushed the fix-4129 branch 2 times, most recently from 2096810 to f48ab0a Compare June 30, 2020 09:40
@efiop efiop changed the title [WIP] azure: use CLI config azure: use CLI config Jun 30, 2020
@efiop efiop merged commit 0275a0d into treeverse:master Jun 30, 2020
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.

azure: use ~/.azure/config

1 participant