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

Update API version to 2024-09-01-preview #14261

Merged

Conversation

bniranjanbhat
Copy link

PR Checklist

Check these items before submitting a PR...

Contribution Guide

Best Practice Guide

  • - Please check this box once you've submitted the PR if you've read through the Contribution Guide and best practices checklist.

Changelog

@@ -3,7 +3,7 @@
"contentVersion": "1.0.0.0",
"parameters": {
"apiVersion": {
"value": "2024-04-01"
"value": "2024-09-01-preview"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API Versions should not be parameterized. Can this (and other instances of this) parameter be removed, and the API version can be hardcoded?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have used API version in the parameters to allow extensibility to the customers.
If we remove this api version now, this would mean a breaking change for consumers at this point.
Would be to possible to allow us take in this in a planned manner in future iterations ?

Copy link
Contributor

@alex-frankel alex-frankel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One change requested. The CI is also not passing.

@alex-frankel alex-frankel merged commit 9d06e39 into Azure:master Dec 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
best practices violations BPA manual validation required This PR requires manual validation readme violations README violations during PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants