Compat: Validate presence of missing connection directive

relay-tools / vscode-apollo-relay

Simple configuration of vscode-apollo for Relay projects.

MIT License

49 stars 13 forks source link

Compat: Validate presence of missing connection directive #26

Closed kastermester closed 4 years ago

kastermester commented 4 years ago

Not sure if this should be a compat only rule or not. I believe one at least encounters proper errors for this under the modern runtime. However when using compat having this rule will aid us in transitioning to modern by being able to use a compat component under modern runtime with higher confidence. Compat components can paginate without @connection directive under classic runtime, so this should help some.

alloy commented 4 years ago

This makes sense to me as a compat rule. Needs a rebase, though.