lukeautry / tsoa

Build OpenAPI-compliant REST APIs using TypeScript and Node
MIT License
3.42k stars 489 forks source link

add ability to include custom swagger extensions in config #1533

Closed gwprice115 closed 4 months ago

gwprice115 commented 7 months ago

All Submissions:

More than happy to write tests and stuff once I get confirmation this is the right approach

Closing issues closes #1532 Put closes #XXXX (where XXXX is the issue number) in your comment to auto-close the issue that your PR fixes.

If this is a new feature submission:

Potential Problems With The Approach

Test plan

TristanSpeakEasy commented 7 months ago

Would love to see this go in will make our lives much easier

WoH commented 6 months ago

This would probably require us to also be able to use a .tsoa.js as config file instead of the tsoa.json, if you don't call generateSwagger directly, but use the cli. (And also live in the spec config) Also, it should be bw-compat if not provided.

gwprice115 commented 5 months ago

This would probably require us to also be able to use a .tsoa.js as config file instead of the tsoa.json, if you don't call generateSwagger directly, but use the cli. (And also live in the spec config) Also, it should be bw-compat if not provided.

gotcha makes sense--would you accept it if it was a static reference similar to how auth is specified?

WoH commented 5 months ago

Yes

github-actions[bot] commented 4 months ago

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days