-
Notifications
You must be signed in to change notification settings - Fork 1
refactor: remove Go SDK code and configuration #54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Remove all Go-related code, configuration, and CI workflows from the repository. The Go SDK is now maintained separately at github.com/tigrisdata/storage-go. Deleted: - go/ directory (all Go source code) - go.mod and go.sum - .github/workflows/go.yaml Updated: - README.md - reference external Go SDK repository - AGENTS.md - remove all Go references Assisted-by: GLM 4.7 via Claude Code
Greptile SummaryThis PR cleanly removes all Go SDK code and configuration from the monorepo as the Go SDK has been moved to a separate repository at https://github.com/tigrisdata/storage-go. Changes:
The refactoring is thorough and complete - no stray Go references remain in the codebase (verified via grep search). The documentation updates properly guide users to the new external repository location. Confidence Score: 5/5
Important Files Changed
|
Greptile's behavior is changing!From now on, if a review finishes with no comments, we will not post an additional "statistics" comment to confirm that our review found nothing to comment on. However, you can confirm that we reviewed your changes in the status check section. This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR". |
Add installation command and documentation reference for the external Go SDK repository. Assisted-by: GLM 4.7 via Claude Code
Assisted-by: GLM 4.7 via Claude Code
Clarify that this monorepo contains JavaScript/TypeScript packages. Assisted-by: GLM 4.7 via Claude Code
- Change title to "Tigris Storage SDKs" - Add "JavaScript/TypeScript SDK" section header - Improve structure to clarify multi-language support Assisted-by: GLM 4.7 via Claude Code
katieschilling
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minting a new version in the monorepo is challenging with the differing version schemes across TS/JS and Go. We have a pointer to a separate go repo in the readme
|
🎉 This PR is included in version 1.0.10 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.1.4 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.11.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
Remove all Go-related code, configuration, and CI workflows from this repository. The Go SDK is now maintained in a separate repository.
Changes
Deleted:
go/directory (all Go source code)go.modandgo.sum.github/workflows/go.yamlUpdated:
README.md- Added reference to external Go SDK repositoryAGENTS.md- Removed all Go-related sections and referencesExternal Go SDK
Users looking for the Go SDK should now use:
Install with:
go get github.com/tigrisdata/storage-goSee the documentation for examples and quickstart advice.
Closes: #53
Note
Removes Go from this repo and consolidates Go development in a separate repository; updates docs accordingly.
go/), module files (go.mod,go.sum), and Go CI workflow (.github/workflows/go.yaml)README.mdto referencetigrisdata/storage-gowith install and docs linksAGENTS.mdby removing Go-specific sections and CI/test/release notes, focusing guidance on JavaScriptWritten by Cursor Bugbot for commit be55cfc. This will update automatically on new commits. Configure here.