Would you consider adding lint for deprecated about eslint-plugin-relay

[eMerzh]

thanks @dotansimha ...
i know those rules indeed,
i was thinking about the client/operation lint like this one https://the-guild.dev/graphql/eslint/rules/no-deprecated

couple reasons why i would hope relayjs-eslint would add this

• an extra dep if like me, you already have this running
• i was hoping this project could take advantage of the rust relay-compiler somehow
• I have issues with the-guild eslint, i can't really set it up 😬 but that's probably me...

[dotansimha]

We do have some rules around deprecations in graphql-eslint - some are from the build-in validations of GraphQL, and some are added by us. You can find it here:

• https://the-guild.dev/graphql/eslint/rules/no-deprecated
• https://the-guild.dev/graphql/eslint/rules/require-deprecation-date
• https://the-guild.dev/graphql/eslint/rules/require-deprecation-reason

It might be useful for you to integrate it, as an additional linting layer.

[eMerzh]

@dotansimha what got me thinking it could [one-day] be possible, is these kinds of changes released in latest relay

facebook/relay@50266e1

[dimaMachina]

If relay-compiler (rust) version has some built-in rules, we can use that and integrate it directly into the IDE using the ESLint extension.

@dotansimha Yes, we can do it as we already do with graphql-js rules

I have issues with the-guild eslint, i can't really set it up 😬 but that's probably me...

@eMerzh there is nothing wrong with graphql-eslint, take a look at working examples

[dotansimha]

• I have issues with the-guild eslint, i can't really set it up 😬 but that's probably me...

feel free to create an issue, we would love to help ;)

• an extra dep if like me, you already have this running

Agree. But I think this might be out of scope for the Relay compiler? as this is a lint issue? 🤔

• i was hoping this project could take advantage of the rust relay-compiler somehow

Actually, that might be interesting. If relay-compiler (rust) version has some built-in rules, we can use that and integrate it directly into the IDE using the ESLint extension. @B2o5T what do you think?