swagger-converter icon indicating copy to clipboard operation
swagger-converter copied to clipboard

Fix yaml serialization through dependency bump.

Open fibsifan opened this issue 3 years ago • 3 comments

Fixes swagger-api/swagger-converter#61

fibsifan avatar Sep 04 '21 11:09 fibsifan

Hi @frantuma @fehguy!

Could you take care of this PR? 🙏

RockyMM avatar Apr 21 '22 12:04 RockyMM

Some time has passed since the PR was created. A newer version 2.13.2 of jackson-jaxrs-yaml-provider has been released since and it probably should be used instead of 2.12.5, but I haven't got the time to test it and could only update the PR. Same goes for swagger-parser (now 2.0.32) and swagger-inflector (2.0.7).

fibsifan avatar Apr 21 '22 13:04 fibsifan

Hm... The solution would be to provide a test case. I already forked this repo (and swagger-parser), but I can't invest time right now to figure out how the tests are organized here. But I did run my fork locally with updated dependencies and it worked for my purposes.

RockyMM avatar Apr 21 '22 16:04 RockyMM