-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix: ensure nestd allOfs are correctly resolved #95
Conversation
Adjust the schema merge logic to include allOf resolution. This ensures that nested allOf's are correctly merged together. Adjust some of the model properties generation to handle some edge cases that were discovered for singleton allOfs. Resolves #94 Signed-off-by: Lucas Roesler <roesler.lucas@gmail.com>
Signed-off-by: Lucas Roesler <roesler.lucas@gmail.com>
...erators/models/testdata/cases/allOf_enum_merging_and_validation/expected/model_connection.go
Show resolved
Hide resolved
Maybe for another PR, but I'm having issues with the dependency: |
I just tested it with my full schema and it works perfect!! Thanks!! |
@LucasRoesler what else is needed to merge? |
Nothing is blocking it, i was just traveling for a few days. Merging now. |
🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.1](https://www.github.com/contiamo/openapi-generator-go/compare/v2.0.0...v2.0.1) (2023-04-08) ### Bug Fixes * ensure nestd allOfs are correctly resolved ([#95](https://www.github.com/contiamo/openapi-generator-go/issues/95)) ([d6605c6](https://www.github.com/contiamo/openapi-generator-go/commit/d6605c6a59c72979cd1c850e444610fcb4f5c8f2)) This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Adjust the schema merge logic to include allOf resolution. This ensures that nested allOf's are correctly merged together.
Adjust some of the model properties generation to handle some edge cases that were discovered for singleton allOfs.
Resolves #94