[typespec-python] Regenerate tests from microsoft/typespec PR #11161 (DO NOT MERGE)#47827
[typespec-python] Regenerate tests from microsoft/typespec PR #11161 (DO NOT MERGE)#47827msyyc wants to merge 1 commit into
Conversation
github-actions
Bot
force-pushed
the
regen/typespec-python-pr-11161
branch
from
6615b32 to
5ea2512
Compare
github-actions
Bot
force-pushed
the
regen/typespec-python-pr-11161
branch
2 times, most recently
from
0a6a269 to
5379a74
Compare
github-actions
Bot
force-pushed
the
regen/typespec-python-pr-11161
branch
from
5379a74 to
c051cf9
Compare
| import geojson | ||
|
|
||
|
|
||
| class Geometry(TypedDict, total=False): |
There was a problem hiding this comment.
These deleted class in types.py are not referred by any model or operation so they should be deleted.
|
|
||
| @overload | ||
| def send(self, body: _types.SendRequest1, *, content_type: str = "application/json", **kwargs: Any) -> None: | ||
| def send(self, body: _types.SendRequest, *, content_type: str = "application/json", **kwargs: Any) -> None: |
There was a problem hiding this comment.
SendRequest1 are referred by many operations which are actually not right since the body of those operations actually refer to different types.
New change includes new SendRequest2/3/.../9 and those new class are referred by related operations which is expected.
| description: str | ||
| """The short, localized friendly description of the operation; suitable for tool tips and detailed | ||
| views.""" | ||
| from .models import CreatedByType, ResourceProvisioningState |
There was a problem hiding this comment.
why are we importing from models here instead of typing using another TypedDict?
There was a problem hiding this comment.
+1 to this question, I'm assuming since these are enums.
| internal only APIs. \"Internal\"""" | ||
|
|
||
|
|
||
| class OperationDisplay(TypedDict, total=False): |
There was a problem hiding this comment.
additionally how do you choose which types to include or not? I'm assuming it's bc they're not actually used in the operations.py file, but do want to double check
There was a problem hiding this comment.
No, these deleted models are not referred by any model or operations.
3 participants
NOTE: This PR exists only to display the code diff. Please do not merge it.
Fixes #47826
Source: microsoft/typespec#11161
Automated regeneration of TypeSpec Python generated tests from microsoft/typespec PR #11161.
This PR was auto-generated. Re-run the workflow to update it after new commits are pushed to the upstream TypeSpec PR