sneak/pixa
1
0
Fork 0
Code Issues 51 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

script/test doesn't follow the REPO_POLICIES conditional-verbose-rerun pattern (always runs -v) #59

New Issue
Open
opened 2026-08-09 03:00:22 +02:00 by clawbot · 0 comments
clawbot commented 2026-08-09 03:00:22 +02:00
Collaborator
Copy Link

REPO_POLICIES.md specifies make test should run without -v first and only rerun with -v on failure, to keep CI/build logs clean on success and detailed on failure (with a go test -race -cover example). Current script/test in this repo always runs go test -timeout 30s -v ./... (now also -race, added in PR #55) unconditionally, with no -cover and no conditional rerun.

Definition of done

  • script/test runs tests without -v first; on failure, reruns with -v for full diagnostic output, then exits non-zero per the pattern documented in REPO_POLICIES.md.
  • Add -cover to the non-verbose run per the policy's Go example.
  • Keep the -race flag added in PR #55.
  • make test still completes well under the 20s budget (30s hard timeout already in place).
  • make check green.
`REPO_POLICIES.md` specifies `make test` should run without `-v` first and only rerun with `-v` on failure, to keep CI/build logs clean on success and detailed on failure (with a `go test -race -cover` example). Current `script/test` in this repo always runs `go test -timeout 30s -v ./...` (now also `-race`, added in PR #55) unconditionally, with no `-cover` and no conditional rerun. ## Definition of done - `script/test` runs tests without `-v` first; on failure, reruns with `-v` for full diagnostic output, then exits non-zero per the pattern documented in `REPO_POLICIES.md`. - Add `-cover` to the non-verbose run per the policy's Go example. - Keep the `-race` flag added in PR #55. - `make test` still completes well under the 20s budget (30s hard timeout already in place). - `make check` green.
clawbot added this to the 1.0.0 milestone 2026-08-09 03:37:54 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No Label
Milestone
No items
No Milestone 1.0.0
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/pixa#59
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?