[Containerregistry-swagger] sdk/resourcemanager/containerregistry/armcontainerregistry #25482
Conversation
This file contains hidden or 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
…1.3.0-beta.4 generation from spec commit: 3855ffb4be0cd4d227b130b67d874fa816736c04
jliusan
requested review from
JiaqiZhang-Dev,
lirenhe and
tadelesh
as code owners
There was a problem hiding this comment.
Pull Request Overview
This pull request updates the Azure Container Registry SDK to a new preview version (v1.3.0-beta.4), synchronizing with updated Swagger specifications. The changes primarily involve updating API reference URLs, removing deprecated task-related functionality, and applying code quality improvements.
Key changes:
- Updated module version to v1.3.0-beta.4
- Updated Swagger spec reference URLs with new commit hash (3855ffb4be0cd4d227b130b67d874fa816736c04)
- Removed task-related clients (Tasks, TaskRuns, Runs, AgentPools) and their supporting code
Reviewed Changes
Copilot reviewed 58 out of 60 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| version.go | Added new version file defining module version v1.3.0-beta.4 |
| go.mod | Updated Azure SDK dependencies to newer versions |
| registries_client_example_test.go | Updated spec URLs and removed 7 task scheduling example functions |
| registries_client.go | Removed GetBuildSourceUploadURL and ScheduleRun methods |
| responses.go | Removed response types for deleted clients (Tasks, TaskRuns, Runs, AgentPools, ScheduleRun) |
| options.go | Removed option types for deleted clients |
| interfaces.go | Removed polymorphic interfaces for task-related types |
| polymorphic_helpers.go | Removed helper functions for task-related polymorphic types |
| fake/server_factory.go | Removed fake server support for deleted clients |
| fake/time_rfc3339.go | Removed unused time handling utilities |
| fake/polymorphic_helpers.go | Removed fake polymorphic helpers for task types |
| fake/*_server.go | Fixed regex match length checks (changed from `matches == nil |
| *_client_example_test.go | Updated spec reference URLs with new commit hash and path structure |
| README.md | Updated example client from AgentPoolsClient to ArchiveVersionsClient |
github-actions
bot
added
the
Mgmt
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
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.
generated base on Containerregistry swagger