Limit uniqueness to `@skip`, `@include` and `@deprecated` directives

At a glance

• Identifier: #471
• Stage: RFC X / Superseded
• Champion: @OlegIlyenko
• Latest activity: 1 commit pushed on 2018-06-25
• Spec PR: https://github.com/graphql/graphql-spec/pull/471

Spec PR description

This proposal directly relates to a discussion in https://github.com/facebook/graphql/issues/429. As was discussed at the latest WG meeting, I'm creating several alternative proposals. This one implements proposed solution 3. Limit the validation to only @skip and @include directives.

It limits the scope of "Directives Are Unique Per Location" to @skip and @include directives only.

This proposal is mutually exclusive with other alternative proposals:

• [RFC] Limit uniqueness to @skip, @include and @deprecated directives (#471, this one)
• [RFC] Limit directive uniqueness to explicitly marked directives (#472)

@leebyron @IvanGoncharov @jjergus I would appreciate your reviews.

Closes #429

Timeline

• Commit pushed on 2018-06-25 by OlegIlyenko: Include @deprecated directive in the validation rule
• Spec PR created on 2018-06-24 by OlegIlyenko
• Commit pushed on 2018-06-24 by OlegIlyenko: Limit uniqueness to @skip and @include directives