-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Open
Labels
MgmtThis issue is related to a management-plane library.This issue is related to a management-plane library.
Description
Python SDK architect provide some suggestions in e-mail RE: Confirmation about Azure MGMT Python SDK release for brownfield service which already converts from Swagger to Typespec about migration doc for Typespec:
We can improve on the descriptions as to why we are making the breaking changes; most of them are (ironically) to avoid future breaking changes; e.g.
* additional_properties captured properties not yet documented. Unfortunately, that had the effect of breaking callers that fished them out from additional_properties when we added documentation for the property.
* flattening had similar problems when new properties were added to what used to be a flattened model.
* the serialize/deserialize ended were not entirely symmetrical leading to hard to debug failures if callers tried to use them to store raw data.
I would not claim that keyword-only arguments give IDEs better intellisense; the reason we make them keyword-only was also because it helped the models evolve over time. Previously, making a parameter optional could cause a breaking change.
Metadata
Metadata
Assignees
Labels
MgmtThis issue is related to a management-plane library.This issue is related to a management-plane library.