-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat(internal/gensnippets): initial commit #3922
Conversation
This tool renderes all examples in the current directory to standalone files. I confirmed this successfully processes every example in this repo. Once I added a go.mod/go.sum in the generated directory, all of the examples passed go vet. A future change will integrate this tool into our automatic generation pipelines and add region tags.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM, just a couple small things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
This tool renders all examples in the current directory to standalone
files.
I confirmed this successfully processes every example in this repo. Once
I added a go.mod/go.sum in the generated directory, all of the examples
passed go vet.
A future change will integrate this tool into our automatic generation
pipelines and add region tags.