vaultik version labels the commit date "build date" #409

Closed
opened 2026-11-10 20:49:43 +01:00 by clawbot · 1 comment
Collaborator

Listed under "Code bugs seen, not fixed" in #399.

Bug. vaultik version prints globals.CommitDate with the label build date (writeVersion in internal/cli/version.go). The value is the date of the commit: the Makefile and the Dockerfile set it from the commit, not from the time of the build.

Definition of done

  • The line reads commit date: followed by the value. The other labels are unchanged, and all values still start in one column.
  • A test that fails on current next checks the line's label and value.
  • make check passes.

Package. internal/cli, file version.go. No README change on next, which does not mention the label; #399 is updated after this merges.

Model: opus-5-5

Listed under "Code bugs seen, not fixed" in https://git.eeqj.de/sneak/vaultik/pulls/399. **Bug.** `vaultik version` prints `globals.CommitDate` with the label `build date` (`writeVersion` in `internal/cli/version.go`). The value is the date of the commit: the `Makefile` and the `Dockerfile` set it from the commit, not from the time of the build. **Definition of done** - The line reads `commit date:` followed by the value. The other labels are unchanged, and all values still start in one column. - A test that fails on current `next` checks the line's label and value. - `make check` passes. **Package.** `internal/cli`, file `version.go`. No README change on `next`, which does not mention the label; https://git.eeqj.de/sneak/vaultik/pulls/399 is updated after this merges. Model: opus-5-5
clawbot self-assigned this 2026-11-10 20:49:43 +01:00
Author
Collaborator

Implemented in #412: the line now reads commit date:, the other labels are padded by one space to keep the values in one column, and a new test in internal/cli/version_test.go checks the commit date: and commit: lines.

Model: opus-5-5

Implemented in https://git.eeqj.de/sneak/vaultik/pulls/412: the line now reads `commit date:`, the other labels are padded by one space to keep the values in one column, and a new test in `internal/cli/version_test.go` checks the `commit date:` and `commit:` lines. Model: opus-5-5
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/vaultik#409