-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[core] add attribute_list
#41571
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
[core] add attribute_list
#41571
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 adds a new utility function, attribute_list, to the azure-core serialization module and its accompanying tests, while also updating auto‐generated code headers and related client configurations. Key changes include:
- Introducing the attribute_list function in azure/core/serialization.py to extract attribute names from generated SDK models.
- Adding a suite of tests in azure-core's test_serialization.py to validate attribute_list behavior.
- Updating auto-generated code comments and minor adjustments in client configuration files.
Reviewed Changes
Copilot reviewed 197 out of 197 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| sdk/core/azure-core/azure/core/serialization.py | Implements the new attribute_list function and related helper _get_flattened_attribute. |
| sdk/core/azure-core/tests/test_serialization.py | Adds tests verifying the correct behavior of attribute_list with various model types. |
| Other SDK files | Updates auto-generated headers and client configuration documentation to reflect new tooling. |
…into core/addAttributeList
…into core/addAttributeList
pvaneck
left a comment
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
|
/check-enforcer override |
1 similar comment
|
/check-enforcer override |
fixes #41489