Skip to content

Fix README.md example usage#982

Merged
deepmap-marcinr merged 1 commit intooapi-codegen:masterfrom
hjblom:patch-2
Mar 14, 2023
Merged

Fix README.md example usage#982
deepmap-marcinr merged 1 commit intooapi-codegen:masterfrom
hjblom:patch-2

Conversation

@hjblom
Copy link
Copy Markdown
Contributor

@hjblom hjblom commented Feb 27, 2023

Currently the example in README.md is not working:

➜  oapi oapi-codegen petstore-expanded.yaml > petstore.gen.go
configuration error: package name must be specified

This can be fixed by providing the required tag for -package

@xieyuschen
Copy link
Copy Markdown

nice job, it lacks such an important hint.

@deepmap-marcinr deepmap-marcinr merged commit 22d1cf4 into oapi-codegen:master Mar 14, 2023
@stevenh
Copy link
Copy Markdown
Contributor

stevenh commented May 5, 2023

This actually isn't needed, the cause was the lack of release after the fix for this went in.

adrianpk pushed a commit to foorester/oapi-codegen that referenced this pull request Jan 16, 2024
adrianpk added a commit to foorester/oapi-codegen that referenced this pull request May 31, 2024
danicc097 pushed a commit to danicc097/oapi-codegen that referenced this pull request Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants