graphql-platform
graphql-platform copied to clipboard
Add analyzers
Product
Hot Chocolate
Is your feature request related to a problem?
Developers using the platform may not be following best practices.
The solution you'd like
Add analyzers such as:
- [ ] Don't use
.With(queryContext)and.Selecttogether. - [x] Node resolver
idargument doesn't need[ID]. - [ ] Don't need
[Service]. - [ ] Somehow warn when the user doesn't have the HC.EF package installed.
- [x] Encourage modern APIs (f.e.
ObjectType<T>). - [ ] Specify requirements when using
[Parent]andQueryContext. - [x] Order of middlewares
- [ ] Not providing an IReadOnlyList<T> keys as the first argument of a dataloader
- [x] Validate that BindMember binds a valid member
- [x] Use source generator for root types aka make class partial
- [x] Parent type check
- [x] Dont use
UseProjectionAttributewhen usingQueryContext<T>
https://github.com/ChilliCream/graphql-platform/pull/8845