-
Notifications
You must be signed in to change notification settings - Fork 311
Updated emitter version #3034
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
Updated emitter version #3034
Conversation
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.
Pull Request Overview
This PR updates the emitter version from 0.22.0 to 0.23.0 and reflects corresponding breaking changes in enum variant naming to align with Rust naming guidelines. The changes rename enum variants from UPPER_CASE to PascalCase format.
Key changes:
- Updated
@azure-tools/typespec-rust
from 0.22.0 to 0.23.0 and@azure-tools/typespec-client-generator-core
from 0.60.0 to 0.60.1 - Renamed enum variants to follow PascalCase convention (e.g.,
KeyType::RSA
→KeyType::Rsa
,EncryptionAlgorithm::RsaOAEP256
→EncryptionAlgorithm::RsaOaep256
) - Updated all test files and benchmarks to use the new enum variant names
Reviewed Changes
Copilot reviewed 6 out of 27 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
eng/emitter-package.json | Updated TypeSpec emitter versions |
sdk/keyvault/azure_security_keyvault_keys/tests/key_client.rs | Updated enum variants in test cases |
sdk/keyvault/azure_security_keyvault_keys/benches/benchmarks.rs | Updated enum variants in benchmark code |
sdk/keyvault/azure_security_keyvault_keys/CHANGELOG.md | Documented breaking changes for enum variant renames |
sdk/keyvault/azure_security_keyvault_certificates/tests/certificate_client.rs | Updated enum variants in certificate tests |
sdk/keyvault/azure_security_keyvault_certificates/CHANGELOG.md | Documented breaking changes for enum variant renames |
Files not reviewed (1)
- eng/emitter-package-lock.json: Language not supported
API Change CheckAPIView identified API level changes in this PR and created the following API reviews azure_security_keyvault_certificates |
No description provided.