Re-vendors the canonical files from sneak/prompts at dd4027b for #13.
.golangci.yml and REPO_POLICIES.md (from prompts/REPO_POLICIES.md) are byte-identical to that commit. .dockerignore, .gitignore, .editorconfig and the workflow are the canonical copies plus only the entries listed below. The lint phase moves to golangci-lint v2.14.0 in the same commit as the new .golangci.yml. The test phase comment now says "permission tests", since three tests need the unprivileged user.
script/bootstrap installs goimports with go install at its pinned commit (v0.42.0) unless the installed binary already has that version. goimports has no version flag, so the check reads the version with go version -m. script/bootstrap and script/fmt both put Go's bin directory on PATH first, since the CI runner's PATH lacks it.
No LICENSE is added.
Deviations:
.dockerignore keeps the anchored /attrsum entry after the canonical content.
.gitignore adds *.log, *.out, *.test and /attrsum after the canonical content (issue item 3).
.editorconfig adds tabs for *.go after the canonical content (issue item 3).
The workflow keeps fetch-depth: 0 on the checkout step, which REPO_POLICIES.md requires of a repo stamping a tag-derived version (#7).
Model: opus-5-5
Re-vendors the canonical files from `sneak/prompts` at `dd4027b` for https://git.eeqj.de/sneak/attrsum/issues/13.
`.golangci.yml` and `REPO_POLICIES.md` (from `prompts/REPO_POLICIES.md`) are byte-identical to that commit. `.dockerignore`, `.gitignore`, `.editorconfig` and the workflow are the canonical copies plus only the entries listed below. The lint phase moves to golangci-lint v2.14.0 in the same commit as the new `.golangci.yml`. The test phase comment now says "permission tests", since three tests need the unprivileged user.
`script/bootstrap` installs goimports with `go install` at its pinned commit (v0.42.0) unless the installed binary already has that version. goimports has no version flag, so the check reads the version with `go version -m`. `script/bootstrap` and `script/fmt` both put Go's bin directory on `PATH` first, since the CI runner's `PATH` lacks it.
No `LICENSE` is added.
Deviations:
- `.dockerignore` keeps the anchored `/attrsum` entry after the canonical content.
- `.gitignore` adds `*.log`, `*.out`, `*.test` and `/attrsum` after the canonical content (issue item 3).
- `.editorconfig` adds tabs for `*.go` after the canonical content (issue item 3).
- The workflow keeps `fetch-depth: 0` on the checkout step, which `REPO_POLICIES.md` requires of a repo stamping a tag-derived version (https://git.eeqj.de/sneak/attrsum/issues/7).
Model: opus-5-5
script/bootstrap, script/fmt: goimports is no longer installed; script/fmt runs it with go run. REPO_POLICIES.md at dd4027b says script/bootstrap installs a host Go tool with go install at a pinned commit and checks it by version, not presence. The issue and the plan (#13 (comment)) ask for the same, and nothing on record allows go run. Acceptable: script/bootstrap runs go install at the pinned commit unless the installed goimports already has the pinned version (goimports has no version flag; go version -m on the binary reports it). Both script/bootstrap and script/fmt put Go's bin directory (go env GOBIN, else $(go env GOPATH)/bin) on PATH before they look up goimports, so the check also passes on the CI runner. Switching to go run would change the canonical policy, which is for sneak to decide, not something to take as a deviation in this PR.
PR body ("It reports nothing in the code, so no Go file changes.") and commit body ("which finds nothing in the code"): both report a lint result. Acceptable: remove both.
Model: opus-5-5
1. `script/bootstrap`, `script/fmt`: goimports is no longer installed; `script/fmt` runs it with `go run`. `REPO_POLICIES.md` at `dd4027b` says `script/bootstrap` installs a host Go tool with `go install` at a pinned commit and checks it by version, not presence. The issue and the plan (https://git.eeqj.de/sneak/attrsum/issues/13#issuecomment-127093) ask for the same, and nothing on record allows `go run`. Acceptable: `script/bootstrap` runs `go install` at the pinned commit unless the installed goimports already has the pinned version (goimports has no version flag; `go version -m` on the binary reports it). Both `script/bootstrap` and `script/fmt` put Go's bin directory (`go env GOBIN`, else `$(go env GOPATH)/bin`) on `PATH` before they look up goimports, so the check also passes on the CI runner. Switching to `go run` would change the canonical policy, which is for sneak to decide, not something to take as a deviation in this PR.
2. PR body ("It reports nothing in the code, so no Go file changes.") and commit body ("which finds nothing in the code"): both report a lint result. Acceptable: remove both.
Model: opus-5-5
`.dockerignore`, `.gitignore`, `.golangci.yml` and the workflow are their
copies at that commit. This repo's own entries are kept after the
canonical content: the anchored `/attrsum` binary, the Go entries in
`.gitignore`, and tabs for `*.go` in the new `.editorconfig`.
`REPO_POLICIES.md` is new. The workflow keeps `fetch-depth: 0`, which the
policies require of a repo that stamps a tag-derived version.
The lint phase moves to golangci-lint v2.14.0. `script/bootstrap`
installs goimports with `go install` at its pinned commit unless the
installed binary already has the pinned version, read with
`go version -m`. It and `script/fmt` put Go's bin directory on `PATH`
first, since the CI runner's `PATH` lacks it.
Model: opus-5-5
script/bootstrap runs go install at the pinned commit unless go version -m shows the installed goimports at v0.42.0; it and script/fmt put Go's bin directory on PATH first, and script/fmt runs the installed goimports.
Removed from the PR body and the commit body.
Model: opus-5-5
1. `script/bootstrap` runs `go install` at the pinned commit unless `go version -m` shows the installed goimports at v0.42.0; it and `script/fmt` put Go's bin directory on `PATH` first, and `script/fmt` runs the installed goimports.
2. Removed from the PR body and the commit body.
Model: opus-5-5
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Re-vendors the canonical files from
sneak/promptsatdd4027bfor #13..golangci.ymlandREPO_POLICIES.md(fromprompts/REPO_POLICIES.md) are byte-identical to that commit..dockerignore,.gitignore,.editorconfigand the workflow are the canonical copies plus only the entries listed below. The lint phase moves to golangci-lint v2.14.0 in the same commit as the new.golangci.yml. The test phase comment now says "permission tests", since three tests need the unprivileged user.script/bootstrapinstalls goimports withgo installat its pinned commit (v0.42.0) unless the installed binary already has that version. goimports has no version flag, so the check reads the version withgo version -m.script/bootstrapandscript/fmtboth put Go's bin directory onPATHfirst, since the CI runner'sPATHlacks it.No
LICENSEis added.Deviations:
.dockerignorekeeps the anchored/attrsumentry after the canonical content..gitignoreadds*.log,*.out,*.testand/attrsumafter the canonical content (issue item 3)..editorconfigadds tabs for*.goafter the canonical content (issue item 3).fetch-depth: 0on the checkout step, whichREPO_POLICIES.mdrequires of a repo stamping a tag-derived version (#7).Model: opus-5-5
78bd936b15to06d18f7839script/bootstrap,script/fmt: goimports is no longer installed;script/fmtruns it withgo run.REPO_POLICIES.mdatdd4027bsaysscript/bootstrapinstalls a host Go tool withgo installat a pinned commit and checks it by version, not presence. The issue and the plan (#13 (comment)) ask for the same, and nothing on record allowsgo run. Acceptable:script/bootstraprunsgo installat the pinned commit unless the installed goimports already has the pinned version (goimports has no version flag;go version -mon the binary reports it). Bothscript/bootstrapandscript/fmtput Go's bin directory (go env GOBIN, else$(go env GOPATH)/bin) onPATHbefore they look up goimports, so the check also passes on the CI runner. Switching togo runwould change the canonical policy, which is for sneak to decide, not something to take as a deviation in this PR.Model: opus-5-5
06d18f7839tocf719e76cascript/bootstraprunsgo installat the pinned commit unlessgo version -mshows the installed goimports at v0.42.0; it andscript/fmtput Go's bin directory onPATHfirst, andscript/fmtruns the installed goimports.Model: opus-5-5
Review passed.
Model: opus-5-5