Conversation
Detect v3 via the "openapi" field, parse servers for host/basePath/schemes, handle schema-based parameters, and convert requestBody to synthetic body parameters. Existing v2 behavior is unchanged.
test.nim: add v2 schema parsing, guessType for inline params, ref resolution, and a negative test for malformed specs. test3.nim: add negative test for malformed v3 spec and server URL with port parsing.
- schema2: Operation tags is [string] not [Tag] (bug) - schema2: SecurityScheme type is the only required field; the rest are conditional on the type value (bug) - schema2: remove duplicate multipleOf in SchemaObject - schema2: Jstring -> JString typo - schema3: Discriminator mapping is a map, not a string - spec: remove duplicate block/break in isKeyword - codegen: remove dead Generator fields types/makeTypes - codegen: fix "may will require" typo - extract shared <~ templates into schemadsl.nim
- genSym: drop named `ident=` parameter to avoid ambiguity with the deprecated two-arg template - newPar: replace with nnkTupleConstr (newPar is removed) - generate template: add explicit import of openapi/codegen so the macro body can resolve newGenerator and friends
- bump to 5.0.0 for nim 2.0+ requirement - CI: nim 2.0 + devel, drop 1.6, switch to jiro4989/setup-nim-action - bundle petstore2.json so tests don't depend on untracked subdir - fix nimble test task to reference correct filenames - --gc:arc -> --mm:arc for nim 2.0
- replace all foreach loops with standard for loops - remove foreach dependency from nimble - CI: use balls test runner, nim 2.0 + devel
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.