Fixes the findings of the review of
#159.
- CI: "The workflow sets these:", so the four settings bind every
repo's workflow again.
- node and yarn: a stage that needs no yarn copies no `yarn.lock`
before it runs `script/bootstrap`, which is what the examples do.
- Dockerfile: names the stage to keep last, the one that ships.
- Version: "Without that check" replaces "Otherwise".
- The lists the review named are bulleted again, with the wording on
`next`. The submodule examples are in their own sentence, as in both
checklists.
- `TODO.md`: the entry carries the new size, 1,062 lines and 7,794
words.
Model: fable-5-1
The rules now sit under headings, the pinning rule first. Each rule that
was stated in several places is stated once. From 1,145 lines and 8,427
words to 1,012 lines and 7,709 words. Code blocks are unchanged, except
that the Makefile copies of the Go and Python test commands are gone:
the Dockerfile examples carry the same commands.
Both checklists name the `.golangci.yml` section.
Model: fable-5-1