Skip to content

Conversation

@vhvb1989
Copy link
Member

@vhvb1989 vhvb1989 commented Aug 4, 2023

By adding the azd-credential to the sdk-client, we will be able to use it for tools like terraform (hashicorp/terraform-provider-azurerm#22827) and kubernetes cli

  • The purpose of this PR is explained in this or a referenced issue.
  • The PR does not update generated files.
  • Tests are included and/or updated for code changes.
  • Updates to CHANGELOG.md are included.
  • MIT license headers are included in each file.

Copy link
Member

@chlowell chlowell left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!

@vhvb1989 vhvb1989 requested a review from chlowell August 8, 2023 17:55
Copy link
Member

@chlowell chlowell left a comment

Choose a reason for hiding this comment

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

I added some test coverage on main. If you rebase onto or merge it, I think this PR can clear the coverage bar now

@scottaddie
Copy link
Member

@vhvb1989 Can you also please add a row to the table at https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/azidentity/README.md#authenticating-via-development-tools?

@chlowell chlowell assigned chlowell and unassigned vhvb1989 Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants