Skip to content

Conversation

@raphael-goetz
Copy link
Contributor

Resolves: #55

Copy link
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 fixes an incorrect Go module import path in the README and removes unused OpenAPI schema definitions from the swagger file.

  • Corrects the Go installation command to use the proper module path
  • Cleans up unused schema definitions from the OpenAPI specification

Reviewed changes

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

File Description
README.md Fixed Go import path from github.com/o7studios/octopus-sdk/sdk/go@latest to the correct module path github.com/o7studios/octopus-sdk@latest
sdk/openapi/octopus.swagger.json Removed unused schema definitions (protobufNullValue, v1Entry, v1EventCall, v1GetResponse, v1ListenRegister, v1Object, v1PageInfo, v1Paginator) that are not referenced in the API specification

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

Copy link
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


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

@raphael-goetz raphael-goetz merged commit 36e21a9 into main Jan 4, 2026
15 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.

bug: false go package docs

2 participants