Skip to content

Fix #18 duplicated directives - #19

Open
Zhenya750 wants to merge 2 commits into
suessflorian:mainfrom
Zhenya750:fix/duplicated-directives
Open

Fix #18 duplicated directives#19
Zhenya750 wants to merge 2 commits into
suessflorian:mainfrom
Zhenya750:fix/duplicated-directives

Conversation

@Zhenya750

@Zhenya750 Zhenya750 commented Aug 17, 2024

Copy link
Copy Markdown

Changes:

  1. Move all print functions to separate file print.go
  2. Add a method introspectionDirectiveDefinition.Equal to equal two directive definitions
  3. Unique directives after getting a schema object. If directives have the same name but different properties, then write directives but log warning

closes #18

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.

Duplicated directives in GraphQL Federation

1 participant