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.

Sign up for GitHub

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

Jump to bottom

Remove "All required parameters must be populated in order to send to Azure." #6064

Closed
lmazuel opened this issue Feb 19, 2025 · 2 comments · Fixed by #6070
Closed

Remove "All required parameters must be populated in order to send to Azure." #6064

lmazuel opened this issue Feb 19, 2025 · 2 comments · Fixed by #6070
Assignees
@msyyc
Labels
emitter:client:python Issue for the Python client emitter: @typespec/http-client-python

Comments

@lmazuel
Copy link
Member

lmazuel commented Feb 19, 2025

Consistent feedback that this sentence is confusing, while no clear indication that it helps anyone. We should just remove it.
@lmazuel lmazuel added the emitter:client:python Issue for the Python client emitter: @typespec/http-client-python label Feb 19, 2025
@msyyc msyyc self-assigned this Feb 20, 2025
@dargilco
Copy link
Member

Please also consider removing "Readonly variables are only populated by the server, and will be ignored when sending a request."

@msyyc
Copy link
Contributor

msyyc commented Feb 20, 2025 •
edited
Loading

Please also consider removing "Readonly variables are only populated by the server, and will be ignored when sending a request."

It's by design to filter the readonly property from request payload and this sentence is just to declare the design idea.

@msyyc msyyc mentioned this issue Feb 20, 2025
Merged
github-merge-queue bot pushed a commit that referenced this issue Feb 20, 2025
@msyyc
[python] remove confusing words from docstring of models (#6070)
18b2c82 
fix #6064
archerzz pushed a commit to archerzz/typespec that referenced this issue Feb 21, 2025
@msyyc
[python] remove confusing words from docstring of models (microsoft#6070
1030b4a 
)

fix microsoft#6064
@dargilco dargilco mentioned this issue Feb 21, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msyyc msyyc

Labels
emitter:client:python Issue for the Python client emitter: @typespec/http-client-python
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[python] remove confusing words from docstring of models microsoft/typespec
3 participants
@lmazuel @dargilco @msyyc