Replies: 3 comments
-
I am all for this change. I think it would be much more maintainable to have only few starters and enable/disable features by properties. Having a separate GraphiQL and Voyager starter for example is a bit wasteful. |
Beta Was this translation helpful? Give feedback.
-
I too am all for this change and agree with @vojtapol feedback - using properties to enable/disable the assortment of tools would be ace. Could also be a good time to re-document all the available application.properties values the framework supports |
Beta Was this translation helpful? Give feedback.
-
Is that done already? |
Beta Was this translation helpful? Give feedback.
-
Over the years the number of starters in this project has grown significantly. Even though they can be categorized in certain groups, the way they act can differ within the category. I'd like to get some feedback on the proposal outlined in this discussion to define a new downsized and improved set of starters.
Proposed reorganization
Deprecate all starters, and create two new starters to hold all functionality.
graphql-kickstart-spring-boot-starter
Will support every use case the current starters support except the testing utilities. It will no longer pull in Spring webflux or webmvc automatically, but instead will require you to do that manually. It will check which web environment type has been loaded and configure that automatically. In case both are available for some reason you can specify which one should be used with a property.
Two possible solutions for defining which technology to use to map the schema: annotation or graphql tools.
The starter will also contain support for all of the above IDEs instead of using all those separate starters. It'll just be a matter of enabling them through properties.
graphql-kickstart-spring-boot-starter-test
Rename of the current test starter, and adding support for webflux too. Spring Boot naming convention for custom starters is to have your library name and then end with "spring-boot-starter". Not sure what the conventions are for test starters like this one.
Beta Was this translation helpful? Give feedback.
All reactions