-
Notifications
You must be signed in to change notification settings - Fork 16
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
spike: embedded code samples rework #1328
base: main
Are you sure you want to change the base?
Conversation
Bump. Is this work still pending or can we close this? @fundthmcalculus |
This was work that needed to be done, but was put on hold per @janpieterz . Happy to have a further discussion about the need for it's completion, but there's still work to do. |
I think we can cancel the work of "all languages". But this pull request might contain some changes that are useful regardless/outside of that. Are there any? |
There is, let me go though it and extract out the useful bits. The goal of this was to automate a lot of the sample generation so it's clearer where to put things. |
protoc-gen-sdk
which creates all endpoints in thedocs/reference/services-generated
folder so you can copy-paste the new endpoints over. feat: Add Docs as a new target trinsic-id/protoc-gen-sdk#55