Use applied directives rather than directives by themselves#72
Use applied directives rather than directives by themselves#72bbakerman merged 1 commit intographql-java:masterfrom
Conversation
|
@enriquedacostacambio any chance we can get this merged if it's okay? This is stopping us from updating to the latest GraphQL tools. Thanks in advance! |
| import jakarta.validation.Payload; | ||
| import java.lang.annotation.Retention; | ||
| import java.lang.annotation.Target; | ||
| import java.util.*; |
There was a problem hiding this comment.
This annoys me - please turn on expand all imports
bbakerman
left a comment
bbakerman
left a comment
There was a problem hiding this comment.
I am happy to accept this PR howev er I would really like it to not use .* imports on the changed files.
Could you please go back and re-format this please
|
Ok progress over perfection - I will let this in and we can fix up the |
Yeah no problem, I'll do that today. Sorry about that, it's the default setting of Intellij and I didn't see it. |
4 participants
Update to use the new
AppliedDirectivesinstead of the oldDirective.This seems to break when upgrading to graphql-java 19, and it does still work for graphql-java 18.