Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for tag v0.2 #1652

Merged
merged 3 commits into from Mar 29, 2022

Conversation

sachindshinde
Copy link
Contributor

@sachindshinde sachindshinde commented Mar 28, 2022

This PR introduces tag v0.2, which adds support for @tag in more locations (arguments, scalars, enums, enum values, input objects, input object fields).

@netlify
Copy link

netlify bot commented Mar 28, 2022

👷 Deploy request for apollo-federation-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit dff6b55

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 28, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@@ -173,7 +189,7 @@ export function typeIncludesDirective(
directiveName: string,
): boolean {
if (isInputObjectType(type)) return false;
const directives = gatherDirectives(type as GraphQLNamedTypeWithDirectives);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably want to remove the special case for isInputObjectType just before this line? (I'll note that this method is only used internally in the case of object types, so doesn't matter so far for our internal usage, but since it's exported ...).

@sachindshinde sachindshinde force-pushed the fed2/tag-v0.2-updates branch 2 times, most recently from f9f2349 to ec0ee3b Compare March 29, 2022 08:06
@sachindshinde sachindshinde merged commit 3a7d833 into apollographql:main Mar 29, 2022
@sachindshinde sachindshinde deleted the fed2/tag-v0.2-updates branch March 29, 2022 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants