Skip to content

Conversation

@simonfelding
Copy link

@simonfelding simonfelding commented Oct 23, 2025

azure-mgmt-batch has some new features I need for my use case.
Here are the breaking changes from v17 to v18:

Breaking Changes
Model DeploymentConfiguration deleted or renamed its instance variable cloud_service_configuration
Model ProxyResource deleted or renamed its instance variable etag
Model Resource deleted or renamed its instance variable location
Model Resource deleted or renamed its instance variable tags
Deleted or renamed model CloudServiceConfiguration
Deleted or renamed method LocationOperations.list_supported_cloud_service_skus

azure-identity has had a lot of updates since v1.17.1. I went through the changelog, and it should be okay. I would like to update it because it has improvements to ManagedIdentityCredential error handling etc, which may be relevant to this plugin.

I'm going to make another PR based on this one soon, so I hope it can be implemented relatively soon. I would have tested this PR locally, but I get an error when I run the tests (#40).

@coderabbitai
Copy link

coderabbitai bot commented Oct 23, 2025

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • pyproject.toml is excluded by !pyproject.toml

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant