-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
dependencies: bump go-azure-sdk
to v0.20240321.1145953
#25332
dependencies: bump go-azure-sdk
to v0.20240321.1145953
#25332
Conversation
Looks like the missing model we need does exist in the Swagger: https://github.com/Azure/azure-rest-api-specs/blob/44a81f85be1e4797fbf5e290fc6b41d48788a6ba/specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/entityTypes/Credential.json#L98-L116 However this isn't being parsed correctly on the Pandora side, so this will want fixing there - notably this model has multiple parents and is located within a sub-directory. |
Fixes: #25314 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the checks pass 👍
Tests look reasonable 👍 |
go-azure-sdk
to v0.20240320.1144505
go-azure-sdk
to v0.20240321.1145953
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Community Note
Description
Updating to sdk version with complete data factory swagger fixes/workarounds
Blocked by changes made in Azure/azure-rest-api-specs#28122