[AutoRelease] t2-hybridkubernetes-2025-03-17-40935(can only be merged by SDK owner) #40089
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR updates the Azure Hybrid Kubernetes Management Client Library to version 1.2.0b2, with an updated API version and enhanced documentation. Key changes include:
- README.md updates: minimum Python version bumped from 3.7+ to 3.8+, section reorganizations, and improved installation/authentication instructions.
- Changelog updates: listing new features, breaking changes, and version bump from 1.2.0b1 to 1.2.0b2.
- Code updates across multiple modules to align with the new API version and improve type hints, import ordering, and asynchronous functionality.
Reviewed Changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| sdk/hybridkubernetes/azure-mgmt-hybridkubernetes/README.md | Updated Python version, reorganized sections, and added installation/authentication instructions. |
| sdk/hybridkubernetes/azure-mgmt-hybridkubernetes/CHANGELOG.md | Detailed new features, breaking changes, and version update to 1.2.0b2. |
| Various client, operations, and configuration modules | Updated API version and improved type hints, policy configuration, and import ordering. |
| sdk/hybridkubernetes/azure-mgmt-hybridkubernetes/generated_samples/* | Updated sample filenames and references to match the new API preview version. |
Comment on lines
+27
to
+28
| By default, [Azure Active Directory](https://aka.ms/awps/aad) token authentication depends on correct configure of following environment variables. | ||
|
|
There was a problem hiding this comment.
Consider updating 'correct configure' to 'correct configuration' for improved clarity.
Suggested change
| By default, [Azure Active Directory](https://aka.ms/awps/aad) token authentication depends on correct configure of following environment variables. | |
| By default, [Azure Active Directory](https://aka.ms/awps/aad) token authentication depends on correct configuration of following environment variables. |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://github.com/Azure/sdk-release-request/issues/5946
Recording test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1727