search

relayjs / eslint-plugin-relay

A plugin for the code linter ESLint to lint specific details about Relay.
MIT License
100 stars 47 forks source link

relay/graphql-naming: Relay no longer enforces fragment name with *_<propName> #107

Open mrtnzlml opened 3 years ago

mrtnzlml commented 3 years ago

See: https://github.com/facebook/relay/commit/ff1c10bc6595f3715f29660b46f779e000be9c70

There is still some enforcement. For example, the module name still has to be included. But the property name is no longer required in the fragment name which makes this line a bit obsolete: https://github.com/relayjs/eslint-plugin-relay/blob/130005b7c1d58fdc2e21da61246735cadfb43999/src/rule-graphql-naming.js#L74

Thanks for having a look! 😊

sibelius commented 3 years ago

I'd like to enforce this using a eslint rule, as simplify a lot