Skip to content

don't remove fragments if they are added by other transforms - #740

Merged
temaEmelyan merged 1 commit into
masterfrom
dont-remove-fragments-if-they-are-added-by-other-transforms
Jul 24, 2026
Merged

don't remove fragments if they are added by other transforms#740
temaEmelyan merged 1 commit into
masterfrom
dont-remove-fragments-if-they-are-added-by-other-transforms

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?

@temaEmelyan temaEmelyan self-assigned this Jul 24, 2026
@temaEmelyan
temaEmelyan requested a review from gnawf July 24, 2026 01:30
@github-actions

Copy link
Copy Markdown

Test Results

  661 files  +  661    661 suites  +661   1m 8s ⏱️ + 1m 8s
2 223 tests +2 223  1 661 ✅ +1 661  562 💤 +562  0 ❌ ±0 
2 231 runs  +2 231  1 669 ✅ +1 669  562 💤 +562  0 ❌ ±0 

Results for commit 543e8ac. ± Comparison against base commit 2a4f8eb.

@temaEmelyan
temaEmelyan merged commit 81a0db2 into master Jul 24, 2026
6 checks passed
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.

2 participants