Skip to content

[graphql] Move graphql_name call to beginning of class

Brett Walker requested to merge bw-graphql-name-position into master

What does this MR do and why?

A change in graphql-ruby gem 1.13 changes how input names are calculated. Having graphql_name called after any arguments are defined can cause naming issues. So just standardize on putting it at the top.

A cop to enforce this will be added !80522 (merged), after this MR gets merged.

Reference issue on GitHub, https://github.com/rmosolgo/graphql-ruby/issues/3919

Related to #210556 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Brett Walker

Merge request reports