-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[DNC] Update version to 2023-05-18-preview, migrate to use AAZ #6554
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
Conversation
❌Azure CLI Extensions Breaking Change Test
|
|
Hi @jackieluc, |
|
Hi @jackieluc, |
|
DNC |
Lost confirmation for aaz cmd. Logged in Azure/azure-cli-dev-tools#414 |
|
@kairu-ms @jsntcy - Looks like the PR automation identified breaking changes. Are there any blockers to this PR from your perspectives that prevents merging? For some additional context, the Also, what is the expected timeline for reviewing and merging changes like this? |
src/dnc/azext_dnc/tests/latest/recordings/test_dnc_controller_create_test_purpose.yaml
Show resolved
Hide resolved
There's no real breaking change in your PR. We will filter out some meaningless messages. |
Thanks @kairu-ms, what is the timeline for something like this to merge? Can you clarify what the milestone means? Is that the target date for this to merge? |
I can merge it anytime. Please fix the CI issues. |
Thanks @kairu-ms, the CI is green now. Please merge if there's no other concerns on your end. |
|
[Release] Update index.json for extension [ dnc ] : https://dev.azure.com/azclitools/internal/_build/results?buildId=77523&view=results |
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az dnc
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally?For new extensions:
About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update
src/index.jsonautomatically.You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify
src/index.json.