-
Notifications
You must be signed in to change notification settings - Fork 882
Adjusting the codegen handler order #2716
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
Comments
Hello! |
@topazur I misunderstood this issue when it was originally opened. We'll get this in for 1.23.0 |
I pushed a PR that I think resolves this issue. But @topazur just so you're aware, we will be deprecating the go-specific configuration options from the codegen request proto in the near future. |
@andrewmbenton Thank you, I will continue to follow the configuration options! |
@topazur after doing some more testing related to extracting our Go codegen as a plugin, I realized I may not understand exactly what is going wrong for you.
Can you expand on this a little bit? Are you saying that inside of your plugin, you don't have access to the configuration options from the built-in go codegen as parsed from But maybe there is another problem? |
Uh oh!
There was an error while loading. Please reload this page.
Version
1.21.0
What happened?
repo: sqlc-gen-go-separate
I have created a plugin with the aim of changing the
codegen/golang
templates to meet my project requirements.But I found that due to the order of the handlers, I cannot use the
config.SQLGen
field in the configuration item.I think we should lower the priority of
golang.Generate
.Playground URL
No response
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go
The text was updated successfully, but these errors were encountered: