Update README to match the new syntax #630
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This patch from 2 weeks ago changed the config syntax: deepmap@c0f382b however, no change was made to the README. This fixes it, already confused few people I assume :) Cheers
|
It took me an hour to notice that types changed to models. Can you also add a migration guide to the top pointing out the big changes ( server-> server of choice and types-> models) I thought it was just a drop in replace for string to object in the configs that I already had. |
|
I am afraid I don't have the context, I do see rename types to models, but I am unsure about the rest. Perhaps someone who has the full picture can write (probably a separate chapter)!? I have just started with this tool so my knowledge is limited. |
adrianpk
pushed a commit
to foorester/oapi-codegen
that referenced
this pull request
Jan 16, 2024
This patch from 2 weeks ago changed the config syntax: deepmap@7d930a4 however, no change was made to the README. This fixes it, already confused few people I assume :) Cheers Co-authored-by: Marcin Romaszewicz <47459980+deepmap-marcinr@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch from 2 weeks ago changed the config syntax:
c0f382b
however, no change was made to the README. This fixes it, already confused few people I assume :)
Cheers