Drop OpenAPI #65
Drop OpenAPI #65
Conversation
There was a problem hiding this comment.
Pull request overview
This PR completely removes OpenAPI/Swagger support from the project by deleting the generated OpenAPI specification file and removing the OpenAPI generation configuration.
Key Changes:
- Removed the OpenAPI plugin configuration from the Buf code generation setup
- Deleted the generated Swagger JSON specification file
- Updated the build system to no longer clean the openapi directory
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| sdk/openapi/octopus.swagger.json | Deleted the generated OpenAPI 2.0 (Swagger) specification file that defined the API schema |
| buf.gen.yaml | Removed the grpc-ecosystem/openapiv2 plugin configuration that was generating the OpenAPI files |
| Makefile | Removed the sdk/openapi directory from the cleanup commands in the generate-proto target |
The changes are consistent and complete for removing OpenAPI generation from the codebase. All three files that referenced OpenAPI have been properly updated - the generated file has been deleted, the generation configuration has been removed, and the build cleanup has been adjusted accordingly.
Note: While not part of this PR, the README.md file on line 8 still mentions "OpenAPI" as one of the generated SDKs, which should be updated in a follow-up change to maintain documentation accuracy.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
# Conflicts: # sdk/openapi/octopus.swagger.json
Resolves: #63