Skip to content

shots: the site's screenshots are the spin release's own now, so brand holds none (R8) - #18

Merged
aledbf merged 1 commit into
mainfrom
shots-from-release
Oct 24, 2026
Merged

aledbf merged 1 commit into
mainfrom
shots-from-release

Conversation

@aledbf

@aledbf aledbf commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Plan step R8, its last part, on brand's side. The companion PR is spin-stack/website#54.

The site's dashboard screenshots now come straight from the spin release the site documents. spin's site-shots.yml puts them on every release as site-shots.tar.gz, and the website's daily sync takes them with the pin that moves to that release. cmd/shots only copied the PNGs and recorded the release, the commit and the test titles, all of which the spin checkout already says. So brand no longer holds the screenshots:

  • removed cmd/shots, shots/, task screenshots, and cmd/shots -check from task check;
  • the laboratory's "The app, as it is" section now points at screens.html (every screen of the app) and at the site's dashboard page;
  • README and the Pages header updated to match.

The website's pinned brand commit still has shots/, but website#54 stops reading it, so the two PRs can merge in either order.

Verified: go tool task check passes (taskgate, sfw check, vet, tests, palette, mark, screens).

🤖 Generated with Claude Code

…d holds none

spin's site-shots.yml takes them of every release and puts them on it as site-shots.tar.gz,
and the website's sync takes them from there with the pin that moves to that release. What
cmd/shots added - the release, the commit and each test's title - is what the spin checkout
says, so the website records it itself. The laboratory points at the catalog, every screen of
the app, and at the site's dashboard page for the documented release's screenshots.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aledbf
aledbf merged commit 31d07eb into main Oct 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant