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
Use go install
instead of go get
in the README example
#434
Comments
go install appeared only in go 1.16, so I believe that the minimum supported version of oapi-codegen would have to be bumped. That being said, I really agree that go install is a better alternative. |
I just noticed the It could be worth at least a mention in the docs something like:
|
@migueleliasweb The version in the As you mention, a conditional advice depending on the go version in the readme seems to be the only way to help users until the last supported version of this project is upgraded to 1.16. I believe @deepmap-marcinr is working on a v2, and is targeting 1.16. |
@migueleliasweb Perhaps it's time to update the manual? |
Agreed, this has caused me some minor confusion today. |
oapi-codegen -package petstore petstore-expanded.yaml > petstore.gen.go error loading swagger spec |
Fixed by #588 , this may be closed |
Thanks folks, you're correct this was closed already |
If anyone stumbles upon this via google, the conventional way now to install this is: |
Hi oapi-codegen team,
If we use
go install github.com/deepmap/oapi-codegen/cmd/oapi-codegen@v1.8.2
instead of the current examplego get github.com/deepmap/oapi-codegen/cmd/oapi-codegen
, Go doesn't addoapi-codegen
as an indirect dependency of the repo. This makes thego.mod
cleaner overall.Would it make sense to change this?
The text was updated successfully, but these errors were encountered: