-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[release/9.0] Update Azure.Identity version #34605
Conversation
1c3d887
to
2730419
Compare
@@ -39,7 +39,7 @@ | |||
<!-- NB: This version affects Visual Studio compatibility. See https://learn.microsoft.com/visualstudio/extensibility/roslyn-version-support --> | |||
<MicrosoftCodeAnalysisVersion>4.8.0</MicrosoftCodeAnalysisVersion> | |||
<MicrosoftCodeAnalysisTestingVersion>1.1.2-beta1.24121.1</MicrosoftCodeAnalysisTestingVersion> | |||
<AzureIdentityVersion>1.11.3</AzureIdentityVersion> | |||
<AzureIdentityVersion>1.12.0</AzureIdentityVersion> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know how often we need to change this, and how other repos are handling the version updates?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least as often as new CVEs are found.
Most repos handle non-dotnet dependency updates manually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dependabot still doesn't work that well for arcade-style repos dependabot/dependabot-core#8490
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this apply to the Cosmos or SQL Server provider?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cosmos only - we have no direct dependency on Azure.Identity on the SQL Server side.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And this is only for tests
No description provided.