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

[WIP] Update the document for client generation #1774

Draft

lirenhe wants to merge 3 commits into main from clientGen-doc-update

Member

lirenhe commented Oct 31, 2024

Note: This PR is still work in progress. Need contributors from different SDK to update the contents.

Fix the grammar issue and leverage tooling to update the wording for the content.
We need to include both snapshot of generated code and how to use generated code in the sample to provide a clear guidance.

Update one sample to remove the deprecated code for TypeSpec and update the .NET sample code.

TODO:

need to refresh all TypeSpec samples and sample code in client.mdx

lirenhe added 3 commits

October 31, 2024 17:54


          Update the document for client generation

44f99c1


          Merge from main branch

0ed4d35


          Fix pettier check error in the doc

efa09d5

markcowl self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

bterlson Awaiting requested review from bterlson bterlson will be requested when the pull request is marked ready for review bterlson is a code owner

markcowl Awaiting requested review from markcowl markcowl will be requested when the pull request is marked ready for review markcowl is a code owner

allenjzhang Awaiting requested review from allenjzhang allenjzhang will be requested when the pull request is marked ready for review allenjzhang is a code owner

timotheeguerin Awaiting requested review from timotheeguerin timotheeguerin will be requested when the pull request is marked ready for review timotheeguerin is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet