Skip to content

Remove HotChocolate.OpenApi#9337

Merged
glen-84 merged 1 commit intomainfrom
gai/remove-hot-chocolate-open-api
Mar 9, 2026
Merged

Remove HotChocolate.OpenApi#9337
glen-84 merged 1 commit intomainfrom
gai/remove-hot-chocolate-open-api

Conversation

@glen-84
Copy link
Copy Markdown
Member

@glen-84 glen-84 commented Mar 9, 2026

Summary of the changes (Less than 80 chars)

  • Remove HotChocolate.OpenApi.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the entire HotChocolate.OpenApi module — a library that provided OpenAPI-to-GraphQL schema translation and query execution. Note that this is distinct from HotChocolate.Adapters.OpenApi, which remains in the codebase and provides different OpenAPI adapter functionality.

Changes:

  • Deletes all source code, tests, resources, snapshots, project files, and the solution file under src/HotChocolate/OpenApi/.
  • Removes the InternalsVisibleTo entry for HotChocolate.OpenApi from HotChocolate.Utilities.csproj and the previously commented-out build entry in Helpers.cs.
  • Removes the TestHotChocolateOpenApi build target from Build.Tests.2.cs.

Reviewed changes

Copilot reviewed 68 out of 69 changed files in this pull request and generated no comments.

File Description
src/HotChocolate/Utilities/src/Utilities/HotChocolate.Utilities.csproj Removes InternalsVisibleTo for the deleted project
.build/Helpers.cs Removes the commented-out OpenApi path entry
.build/Build.Tests.2.cs Removes the TestHotChocolateOpenApi build target
src/HotChocolate/OpenApi/** (all files) Complete deletion of the OpenApi module source, tests, resources, snapshots, build props, and solution file
Files not reviewed (1)
  • src/HotChocolate/OpenApi/src/HotChocolate.OpenApi/Properties/OpenApiResources.Designer.cs: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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