Cleaner interface - Githubissues

jakehschwartz / finatra-swagger

Swagger Support For Finatra

https://jakehschwartz.github.io/swagger-finatra

Apache License 2.0

28 stars 19 forks source link

Cleaner interface #10

Open jakehschwartz opened 7 years ago

jakehschwartz commented 7 years ago

Clean up the interface a bit:

Change SwaggerController to DocsController
Move controllers to controller package
More

jakehschwartz commented 7 years ago

I don't like the fact that there are 2 controllers to import, will merge WebjarsController into DocsController

jakehschwartz commented 7 years ago

Nevermind, I'll leave this for now

jakehschwartz commented 7 years ago

Changed my mind again. The main bonuses are ease of use (only having 1 controller) and more Finatra like (having Swagger be managed through DI)

jakehschwartz commented 7 years ago

This is cleaner than it was, but it would be nice not have to inject the Swagger object in all the controllers just so it can be assigned to an implicit.