Skip to content
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

[engsys] remove unused dependency url #24380

Merged
merged 3 commits into from Jan 6, 2023

Conversation

jeremymeng
Copy link
Contributor

Related: #24379

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

The negative LRO tests have been updated to have proper key identifier format.
Previously `url` is more forgiving on invalid urls.
@jeremymeng
Copy link
Contributor Author

/azp run js - keyvault-certificates - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jeremymeng
Copy link
Contributor Author

/azp run js - keyvault-secrets - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jeremymeng
Copy link
Contributor Author

/azp run js - keyvault-keys - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jeremymeng
Copy link
Contributor Author

Pipeline failure due to an existing flaky test that has been failing for a while.

@jeremymeng
Copy link
Contributor Author

/check-enforcer override

@jeremymeng jeremymeng merged commit 95705cb into Azure:main Jan 6, 2023
@jeremymeng jeremymeng deleted the engsys/remove-unused-url-dep branch January 6, 2023 21:38
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this pull request Jun 9, 2023
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.

None yet

4 participants