Remove unused schema definitions (e.g. `--remove-unreferenced-schema` ) about refitter HOT 7 CLOSED

Will be looking at it.

This is mainly useful for people like me/company where we filter endpoints and only want the minimum required OpenApi spec parts in an assembly/module.

Reason being that breaking changes in contracts become easier to manage, due to less clutter after re-generating the endpoints and contracts.

from refitter.

Is excludedTypeNames a possible alternative?
I have crappy OpenAPI with unreferenced schemas. I tried to get rid of the types with ExcludedTypeNames but it doesn't seem to have any effect (I tried the schema name, namespace-prefixed class name... nothing worked).

@christianhelle
As we have includePathMatches, an equivalent includeSchemaMatches might be an option to include unreferenced schemas?

from refitter.

As we have includePathMatches, an equivalent includeSchemaMatches might be an option to include unreferenced schemas?

i like that suggestion, it's easy to implement and reason about

from refitter.

@kirides Thanks for taking the time to investigate and suggest this

This could be really interesting as an option but definitely not as the default behavior. I have on multiple occasions used an OpenAPI spec for the sole purpose of generating contracts

Looks like you already have the code, if you create a pull request and all the tests pass then I'll approve so can get this in 😄

from refitter.

@all-contributors please add @jods4 for ideas

from refitter.

@christianhelle

I've put up a pull request to add @jods4! 🎉

from refitter.

I was gonna say don't bother, but you're too quick!
Thanks for the feature!

from refitter.