Skip to content

{Service Connector}: Fix auth type issue and check db existence - #6625

Merged
kai ru (kairu-ms) merged 3 commits into
Azure:mainfrom
xfz11:svc/fix
Aug 14, 2023
Merged

kai ru (kairu-ms) merged 3 commits into
Azure:mainfrom
xfz11:svc/fix

Conversation

@xfz11

@xfz11 xfz11 commented Aug 10, 2023 •

Copy link
Copy Markdown
Contributor

This checklist is used to make sure that common guidelines for a pull request are followed.
Fix issue Azure/azure-cli#27134

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

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.json automatically.
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.

@azure-client-tools-bot-prd

azure-client-tools-bot-prd Bot commented Aug 10, 2023 •

Copy link
Copy Markdown
️✔️Azure CLI Extensions Breaking Change Test
️✔️Non Breaking Changes

@yonzhan

Copy link
Copy Markdown
Collaborator

Service Connector

@xfz11 xfz11 changed the title {Service Connector}: Fix auth type code issue {Service Connector}: Fix auth type issue and check resource existence Aug 11, 2023
@xfz11 xfz11 changed the title {Service Connector}: Fix auth type issue and check resource existence {Service Connector}: Fix auth type issue and check db existence Aug 11, 2023
@xfz11

xfz11 commented Aug 14, 2023

Copy link
Copy Markdown
Contributor Author

kai ru (@kairu-ms) Could you help merge the PR?

@kairu-ms
kai ru (kairu-ms) merged commit 89a372d into Azure:main Aug 14, 2023
@xfz11 xfz11 mentioned this pull request Aug 14, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants