pb33f / openapi-changes

The world's sexiest OpenAPI breaking changes detector. Discover what changed between two OpenAPI specs, or a single spec over time. Supports OpenAPI 3.1, 3.0 and Swagger
https://pb33f.io/openapi-changes/
Other
180 stars 16 forks source link

dont consider unused removed schemas as a breaking change #125

Closed teto closed 6 months ago

teto commented 6 months ago

first of all thank you for the amazing tool, I am excited to share it with some friends. I tested on an openapi.json generated from our code, our openapi.json contains many unused schemas that we try to remove to clean the API. So openapi-changes considers

- Schemas Removed {X}

as a breaking change, which is LGTM if the schema is used but if it's not referenced anywhere, I would think it's inconsequential ?

teto commented 6 months ago

ha I misread the output, the schemas removed were indeed used. Sry for the noise