Repository navigation
chore: migrate to proto-tap org - #37
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The migration changes are consistent across code, release configuration, and documentation, with no unresolved blocking findings.
No open issues 🎉
1 file not reviewed
gen/tap/v1/tap.pb.go: Generated file
What changed in this PR
Migrates sql-tap to the proto-tap organization, aligning Go package paths, release publishing, and installation documentation with the repository transfer.
Changes:
- Updates the module path, internal imports, generated protobuf metadata, and tooling configuration.
- Adds Homebrew cask publishing under
proto-tap, retaining a temporary migration formula in the old tap and removing unused Nix publishing. - Refreshes installation instructions and project links, and removes repository-level funding configuration in favor of organization defaults.
| File | Description |
|---|---|
web/web.go |
Updates web server imports. |
web/web_test.go |
Updates web test imports. |
tui/timeline.go |
Updates timeline imports. |
tui/model.go |
Updates TUI model imports. |
tui/list.go |
Updates query list imports. |
tui/inspector.go |
Updates inspector imports. |
tui/format.go |
Updates formatting imports. |
tui/filter.go |
Updates filter imports. |
tui/filter_test.go |
Updates filter test imports. |
tui/export.go |
Updates export imports. |
tui/export_test.go |
Updates export test imports. |
tui/explain.go |
Updates EXPLAIN view imports. |
tui/editor.go |
Updates editor imports. |
tui/analytics.go |
Updates analytics imports. |
server/server.go |
Updates gRPC server imports. |
server/server_test.go |
Updates server test imports. |
README.md |
Updates installation paths and documents Homebrew migration. |
query/normalize_test.go |
Updates normalization test imports. |
query/bind_test.go |
Updates parameter binding test imports. |
proxy/postgres/proxy.go |
Updates PostgreSQL proxy imports. |
proxy/postgres/proxy_test.go |
Updates PostgreSQL integration test imports. |
proxy/postgres/export_test.go |
Updates test helper imports. |
proxy/postgres/conn.go |
Updates PostgreSQL connection imports. |
proxy/postgres/conn_test.go |
Updates PostgreSQL connection test imports. |
proxy/mysql/proxy.go |
Updates MySQL proxy imports. |
proxy/mysql/proxy_test.go |
Updates MySQL integration test imports. |
proxy/mysql/conn.go |
Updates MySQL connection imports. |
main.go |
Updates CLI entry-point imports. |
go.mod |
Changes the Go module identity. |
gen/tap/v1/tap.pb.go |
Updates embedded protobuf Go package metadata. |
flake.nix |
Updates the package homepage. |
explain/explain_test.go |
Updates EXPLAIN test imports. |
explain/build_args_test.go |
Updates argument-building test imports. |
dsn/dsn_test.go |
Updates DSN test imports. |
detect/detect_test.go |
Updates detection test imports. |
config/config_test.go |
Updates configuration test imports. |
cmd/sql-tapd/main.go |
Updates daemon entry-point imports. |
cmd/sql-tapd/main_test.go |
Updates daemon test imports. |
clipboard/clipboard_test.go |
Updates clipboard test imports. |
ci/ci.go |
Updates the generated API import. |
ci/ci_test.go |
Updates CI client test imports. |
buf.gen.yaml |
Changes the generated Go package prefix. |
broker/broker.go |
Updates broker imports. |
broker/broker_test.go |
Updates broker test imports. |
.goreleaser.yaml |
Adds cask publishing, retains a migration formula, and removes Nix publishing. |
.golangci.yaml |
Updates the local import prefix. |
.github/workflows/release.yml |
Supplies the legacy tap publishing token. |
.github/FUNDING.yaml |
Removes the repository funding override. |
Review effort: Balanced
This was referenced Oct 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
sql-tap has been transferred from
mickamyto theproto-taporg. This PR updates the module path, the release pipeline, and the docs to match, and switches Homebrew distribution from a formula inmickamy/tapto a cask inproto-tap/tap.Changes
Module path (
refactor: change module path to github.com/proto-tap/sql-tap)go.mod, all imports,buf.gen.yaml(go_package_prefix),.golangci.yaml(local-prefixes) now usegithub.com/proto-tap/sql-tapgen/regenerated; the only diff is the embeddedgo_package(plugin versions kept as-is)Release pipeline (
build: publish homebrew cask to proto-tap/tap)homebrew_caskspublishingCasks/sql-tap.rbtoproto-tap/homebrew-tapwithTAP_GITHUB_TOKEN(org secret)sql-tapandsql-tapdhooks.post.install_steps(postflight_steps, requires GoReleaser v2.19+)brewsfor this release only, still publishingFormula/sql-tap.rbtomickamy/homebrew-tap, now withMICKAMY_GITHUB_TOKENdisable! date: "2027-04-27", replacement_cask: "proto-tap/tap/sql-tap": existing users get a deprecation warning until the date, then the formula is disabled and points to the caskcaveatsexplain how to switch (brew uninstall sql-tap→brew install --cask proto-tap/tap/sql-tap)nix:(the targetmickamy/nur-packagesnever existed, so it was a no-op)MICKAMY_GITHUB_TOKENto GoReleaser inrelease.ymlDocs (
docs: update install instructions for proto-tap)brew install --cask proto-tap/tap/sql-tapwith a migration note formickamy/tapusers, and new paths forgo install,git clone, Nix, Docker, and the grpc-tap linkflake.nix: updatehomepageFunding (
chore: remove FUNDING.yaml in favor of org-level config)proto-tap/.github/FUNDING.yml, which applies to all org reposVerification
go build ./...,go vet ./...,go test -race ./...passgoreleaser release --snapshot --clean --skip=publish(v2.19.0) generates the expected cask and formulagoreleaser checkexits non-zero becausebrewsis deprecated; this is expected and CI doesn't run it. The release itself only warns.After merge
proto-tap/homebrew-tapand the formula inmickamy/homebrew-taphasdisable!brewsandMICKAMY_GITHUB_TOKENgo install github.com/proto-tap/sql-tap@latestonly works once a tag with the new module path is publishedRelated
flake.nixvendorHashis still a placeholder, out of scope here)