Skip to content

fix: fix selecting parent of unique nodes#2594

Closed
alepane21 wants to merge 2 commits intomainfrom
sergiy/eng-9097-fix-wrong-external-parent-selection-when-use-interfaces-2
Closed

fix: fix selecting parent of unique nodes#2594
alepane21 wants to merge 2 commits intomainfrom
sergiy/eng-9097-fix-wrong-external-parent-selection-when-use-interfaces-2

Conversation

@alepane21
Copy link
Copy Markdown
Contributor

@alepane21 alepane21 commented Mar 5, 2026

Summary by CodeRabbit

  • Chores
    • Updated internal dependencies to maintain compatibility and stability.

Checklist

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1f854234-9924-4a15-9085-6b3fa4545cfa

📥 Commits

Reviewing files that changed from the base of the PR and between 177a1c0 and 7459c47.

⛔ Files ignored due to path filters (2)
  • router-tests/go.sum is excluded by !**/*.sum
  • router/go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • router-tests/go.mod
  • router/go.mod

Walkthrough

This PR updates the github.com/wundergraph/graphql-go-tools/v2 dependency version from v2.0.0-rc.257 to v2.0.0-rc.259.0.20260305233335-726700149835 in both router and router-tests modules.

Changes

Cohort / File(s) Summary
Dependency Version Bumps
router/go.mod, router-tests/go.mod
Updated github.com/wundergraph/graphql-go-tools/v2 from v2.0.0-rc.257 to v2.0.0-rc.259.0.20260305233335-726700149835 across both modules.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims to fix selecting parent of unique nodes, but the actual changes only update dependency versions in go.mod files with no implementation changes. Update the PR title to accurately reflect the actual changes, such as 'chore: bump graphql-go-tools/v2 dependency' or provide the actual code changes for the stated fix.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 5, 2026

❌ Internal Query Planner CI checks failed

The Internal Query Planner CI checks failed in the celestial repository, and this is going to stop the merge of this PR.
If you are part of the WunderGraph organization, you can see the PR with more details.

@devsergiy devsergiy closed this Mar 6, 2026
@devsergiy devsergiy deleted the sergiy/eng-9097-fix-wrong-external-parent-selection-when-use-interfaces-2 branch March 6, 2026 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants