Fix include definitions subfolders

Redocly / swagger-repo

CLI tool to help you manage your OpenAPI(fka Swagger) repo.

MIT License

18 stars 29 forks source link

Closed alexsotocx closed 6 years ago

alexsotocx commented 6 years ago

With this fix you will be able to: Inside the definitions folders you can create multiple folders to split the documentation.

For example:

/Definitions -/Definitions/v1/Some.yaml -/Definitions/v2/Some.yaml

In the past you will get There is already a defined collection.

RomanHotsiy commented 6 years ago

Hey @alexsotocx, Thanks for the PR. But I can't merge this as it's a breaking change. There are users who rely on the current way it works.

But I think this behavior may be useful if we:

put into subfolders if only there is a conflict. It is not so good idea to make "folders" in definitions. Swagger in some cases relies on this to be flat.
put it behind some flag e.g. --nest-if-nameclash (or something better)

Could you update your PR plz to address those?