upgrade swagger-parser to latest version#18436
Conversation
|
thanks for the PR. i'm looking into the gradle test issues updating gradle to latest 7.x and 8.x didn't help as shown in #18300 i'll try to contact Gradle team to see if they can help. |
50b1b44 to
ba6b98c
Compare
ba6b98c to
ad1c88c
Compare
|
Tests are still failing with this error. It could be a gradle bug but it feels like something else; note that the jackson version in the error message is |
66abe49 to
09949e9
Compare
09949e9 to
51a2a5b
Compare
|
Found some additional places to upgrade gradle versions and all checks are passing now. @wing328 could you take a look? |
|
nice 👍 i'll take another later today |
|
looks good. let's give it a try |
|
saw the following in travis ci: full log: https://app.travis-ci.com/github/OpenAPITools/openapi-generator/builds/270381537 can you please take a look when you've time? |
This upgrades swagger-parser to the latest release. In manual testing, this upgrade restored the generator's ability to resolve external references in OpenAPI specs.
Fixes #18429
PR checklist
Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*.IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
master(upcoming 7.1.0 minor release - breaking changes with fallbacks),8.0.x(breaking changes without fallbacks)