Move the Go module to sneak.berlin/go/keyfunc (closes #15) #41

Merged
clawbot merged 1 commits from issue-15-module-path into next 2026-10-04 01:42:45 +02:00
Collaborator

Moves the Go module to sneak.berlin/go/keyfunc, as decided on #15.

  • go.mod, every import of git.eeqj.de/sneak/keyfunc/... and the -X path in the Makefile LDFLAGS now use the new path. The old path gets no alias or fallback. git.eeqj.de/sneak/secret is a different module and is unchanged.
  • The import lines changed position only because make fmt sorts each import group by path, so sneak.berlin/... now comes after github.com/... and golang.org/....
  • README Getting Started gives go install sneak.berlin/go/keyfunc/cmd/keyfunc@latest next to the build from a clone. This issue's line leaves the TODO list.
  • Rebased onto next, which carries #37: the TODO list now names the open 1.0 issues in place of the license line.

Disclosures:

  • @latest resolves only once main carries the move. Until then the install line works only with an explicit commit of this branch.
  • Deviation: the go install check against this branch's commit fetched keyfunc directly from the repo, but fetched its dependencies through the default module proxy rather than with GOPROXY=direct. github.com/tyler-smith/go-bip39 no longer exists on GitHub and is only served by the proxy. That was already true under the old path.

Model: opus-5-5

Moves the Go module to `sneak.berlin/go/keyfunc`, as decided on https://git.eeqj.de/sneak/keyfunc/issues/15. - `go.mod`, every import of `git.eeqj.de/sneak/keyfunc/...` and the `-X` path in the `Makefile` `LDFLAGS` now use the new path. The old path gets no alias or fallback. `git.eeqj.de/sneak/secret` is a different module and is unchanged. - The import lines changed position only because `make fmt` sorts each import group by path, so `sneak.berlin/...` now comes after `github.com/...` and `golang.org/...`. - README Getting Started gives `go install sneak.berlin/go/keyfunc/cmd/keyfunc@latest` next to the build from a clone. This issue's line leaves the TODO list. - Rebased onto `next`, which carries https://git.eeqj.de/sneak/keyfunc/pulls/37: the TODO list now names the open 1.0 issues in place of the license line. Disclosures: - `@latest` resolves only once `main` carries the move. Until then the install line works only with an explicit commit of this branch. - Deviation: the `go install` check against this branch's commit fetched keyfunc directly from the repo, but fetched its dependencies through the default module proxy rather than with `GOPROXY=direct`. `github.com/tyler-smith/go-bip39` no longer exists on GitHub and is only served by the proxy. That was already true under the old path. Model: opus-5-5
clawbot added the needs-review label 2026-10-03 14:13:00 +02:00
clawbot self-assigned this 2026-10-03 14:13:00 +02:00
clawbot added needs-rebase and removed needs-review labels 2026-10-03 14:43:28 +02:00
clawbot added 1 commit 2026-10-03 15:11:35 +02:00
REPO_POLICIES.md sets the Go module root to sneak.berlin/go/<name>.
go.mod, every import of the old path and the -X path in the Makefile
LDFLAGS now use sneak.berlin/go/keyfunc; make fmt moved those imports
to their new place in the sort order. The old path keeps no alias. The
README gives the go install line next to the build from a clone and
drops this issue from its TODO list, which now names the open 1.0
issues.

Model: opus-5-5
clawbot force-pushed issue-15-module-path from 72dd386bbf to 4e1258302f 2026-10-03 15:11:35 +02:00 Compare
clawbot added needs-review and removed needs-rebase labels 2026-10-03 15:11:38 +02:00
Author
Collaborator
  • PR body, the last sentence of the third bullet and the "Conflict ahead" disclosure: both describe the branch as it was before the rebase onto next. They say the license line (#14) is still in the README TODO list and that a rebase against #37 is still to come. On the branch, the license line is gone, the rebase is done, and the list names #38, #39, #40 and #42. The body also contradicts its own fourth bullet. Acceptable: delete that sentence and that disclosure, so the body describes only the rebased branch.

Model: opus-5-5

- PR body, the last sentence of the third bullet and the "Conflict ahead" disclosure: both describe the branch as it was before the rebase onto `next`. They say the license line (https://git.eeqj.de/sneak/keyfunc/issues/14) is still in the README TODO list and that a rebase against https://git.eeqj.de/sneak/keyfunc/pulls/37 is still to come. On the branch, the license line is gone, the rebase is done, and the list names https://git.eeqj.de/sneak/keyfunc/issues/38, https://git.eeqj.de/sneak/keyfunc/issues/39, https://git.eeqj.de/sneak/keyfunc/issues/40 and https://git.eeqj.de/sneak/keyfunc/issues/42. The body also contradicts its own fourth bullet. Acceptable: delete that sentence and that disclosure, so the body describes only the rebased branch. Model: opus-5-5
clawbot added needs-rework and removed needs-review labels 2026-10-03 17:44:42 +02:00
Author
Collaborator

Rework: the description now describes only the rebased branch; the stale sentence in the third bullet and the "Conflict ahead" disclosure are deleted. No code changed.

Model: opus-5-5

Rework: the description now describes only the rebased branch; the stale sentence in the third bullet and the "Conflict ahead" disclosure are deleted. No code changed. Model: opus-5-5
clawbot added needs-review and removed needs-rework labels 2026-10-03 17:59:24 +02:00
Author
Collaborator

Review passed.

Model: opus-5-5

Review passed. Model: opus-5-5
clawbot merged commit 8d1c873bb7 into next 2026-10-04 01:42:45 +02:00
clawbot deleted branch issue-15-module-path 2026-10-04 01:42:45 +02:00
Sign in to join this conversation.