Skip to content

[Identity] Readme updates#12310

Merged
sophiajt merged 3 commits into
Azure:masterfrom
sophiajt:readme_updates
Nov 5, 2020
Merged

[Identity] Readme updates#12310
sophiajt merged 3 commits into
Azure:masterfrom
sophiajt:readme_updates

Conversation

@sophiajt
Copy link
Copy Markdown
Contributor

@sophiajt sophiajt commented Nov 5, 2020

This should align the README with other languages.

Fixes #10945


### Authenticate the client

When debugging and executing code locally it is typical for a developer to use their own account for authenticating calls to Azure services. There are several developer tools which can be used to perform this authentication in your development environment.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like this!

Comment thread sdk/identity/identity/README.md Outdated
### Environment variables
## Environment Variables

[`DefaultAzureCredential`][ref_defaultazurecredential] and [`EnvironmentCredential`][ref_environmentcredential] can be configured with environment variables. Each type of authentication requires values for specific variables:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do ref_defaultazurecredential and ref_environmentcredential exist? I couldn't find them elsewhere in this file 🤔

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.

Ooops good catch. I didn't copy/paste those

Copy link
Copy Markdown
Contributor

@sadasant sadasant left a comment

Choose a reason for hiding this comment

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

Looks good! 🌞 thank you!

@sophiajt sophiajt merged commit ce71fb5 into Azure:master Nov 5, 2020
@sophiajt sophiajt deleted the readme_updates branch November 5, 2020 23:51
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.

[Identity] Docs review README changes

2 participants