graphql-dotnet / graphql-dotnet/graphql-dotnet

Roslyn analyzers

Open
#3,732 14 comments 1 reaction 0 assignees View on GitHub
analyzers
Dominant language
C#
Stars
6k
Forks
945
Avg merge
3d 13h
Merged PRs (30d)
14

Description

This is an aggregating issue for Roslyn analyzers, code refactorings, and source generators. The list will be updated with additional suggestions.

# Analyzers+CodeFixes
- [x] Rewrite the obsolete `Field().Name("name")` with `Field("name")` - #3697
- [x] Rewrite the obsolete `Field(name, description, ...)` with `Field(name).Description(description)...` - #3697
- [x] Forbid using resolvers on the non-output graph types - #3697
- [x] Ensure an input field can be mapped to the source type field - #3720
- [x] Rewrite the obsolete `Connection().Name("name")` with `Connection("name")` - #3747
- [x] Rewrite the obsolete `Argument` with `Argument` - #3755
- [x] Detect awaitable resolvers when used in sync `Resolve*` methods and rewrite with `Resolve*Async` - #3763
- [ ] Ensure all fields defined by the interface are implemented by the graph type

# CodeRefactoring
- [ ] Suggest replacing `Arguments(new QueryArgument(), ...)` method with a chain of `.Argument()` calls (and back?)

# Source generation
- [ ] Rewrite `ObjectExtensions.ToObject`, which uses reflection with a source generator - #3757

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.