Skip to content

Wait for started node and entity resolvers before completing nodes and _entities - #10735

Merged
tobias-tengler merged 1 commit into
mainfrom
tte/nodes-await-started-resolvers
Oct 28, 2026
Merged

tobias-tengler merged 1 commit into
mainfrom
tte/nodes-await-started-resolvers

Conversation

@tobias-tengler

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings October 28, 2026 09:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@tobias-tengler
tobias-tengler merged commit 704f20f into main Oct 28, 2026
158 checks passed
@tobias-tengler
tobias-tengler deleted the tte/nodes-await-started-resolvers branch October 28, 2026 09:19
@github-actions

Copy link
Copy Markdown
Contributor

Patch coverage

77.0% of changed lines covered (77/100)

File Covered Changed Patch %
…/Core/src/Types/Types/Relay/NodeFieldResolvers.cs 37 51 72.5% 🔴
…/src/ApolloFederation/Resolvers/EntitiesResolver.cs 40 49 81.6% 🟡
Uncovered changed lines (JSON)
{
  "sha": "06ca614b30056e2d19d4e103f6f426f887912fca",
  "files": [
    { "path": "src/HotChocolate/Core/src/Types/Types/Relay/NodeFieldResolvers.cs", "ranges": [[96, 97], [99, 100], [127, 128], [134, 141]] },
    { "path": "src/HotChocolate/ApolloFederation/src/ApolloFederation/Resolvers/EntitiesResolver.cs", "ranges": [[75, 78], [86, 90]] }
  ]
}

Project coverage: 59.2% (318131/537339 lines)

glen-84 pushed a commit that referenced this pull request Oct 28, 2026
glen-84 pushed a commit that referenced this pull request Oct 28, 2026
glen-84 pushed a commit that referenced this pull request Oct 28, 2026
glen-84 pushed a commit that referenced this pull request Oct 28, 2026
This was referenced Oct 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants