Skip to content

Remove all document variables hint#743

Open
temaEmelyan wants to merge 4 commits into
masterfrom
remove-all-document-variables-hint
Open

Remove all document variables hint#743
temaEmelyan wants to merge 4 commits into
masterfrom
remove-all-document-variables-hint

Conversation

@temaEmelyan

Copy link
Copy Markdown
Member

Please make sure you consider the following:

  • Add tests that use __typename in queries
  • Does this change work with all nadel transformations (rename, type rename, hydration, etc)? Add tests for this.
  • Is it worth using hints for this change in order to be able to enable a percentage rollout?
  • Do we need to add integration tests for this change in the graphql gateway?
  • Do we need a pollinator check for this?

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

Test Results

  663 files  +  663    663 suites  +663   45s ⏱️ +45s
2 228 tests +2 228  1 665 ✅ +1 665  563 💤 +563  0 ❌ ±0 
2 236 runs  +2 236  1 673 ✅ +1 673  563 💤 +563  0 ❌ ±0 

Results for commit 4f53635. ± Comparison against base commit 58c419e.

♻️ This comment has been updated with latest results.

@temaEmelyan
temaEmelyan force-pushed the remove-all-document-variables-hint branch from 04874c0 to 3024ecb Compare July 27, 2026 08:42
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.

1 participant