Skip to content

Carry the Tap family icon: README title, book favicon; drift check v6 - #85

Merged
tap merged 1 commit into
mainfrom
ccr-7fe10311-lqzred
Oct 8, 2026
Merged

tap merged 1 commit into
mainfrom
ccr-7fe10311-lqzred

Conversation

@tap

@tap tap commented Oct 8, 2026

Copy link
Copy Markdown
Owner

What this changes

MuTap's mark from TapHouse brand/ now appears in two places:

  • README title: .github/icon-light.svg and icon-dark.svg, switched by the reader's GitHub theme.
  • The book's favicon: book/theme/favicon.svg and favicon.png.

style.yml pins the drift check at TapHouse v6 with icon: MuTap, so all four files are guarded byte for byte from now on.

Why

This is the family icon rollout (tap/TapHouse#10, #11). The files come from sync.sh --icon MuTap at v6. The shared configs are unchanged since v5.

Verification

  • Drift check: I ran TapHouse v6's drift-check step locally against this branch. It passes.
  • Book: it builds with mdBook v0.4.40, the version docs.yml pins. Compared with a build without the two files, only the two favicons differ; every other file is byte-identical.
  • README title: GitHub renders it on a sibling repo's branch (AmbiTap), with both sources resolving to that branch's files; this one uses the same markup. The title's anchor moves from #mutap to #-mutap, and nothing in the repo links to it.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JJQ9mTiYXTVqyTMKhoScnP


Generated by Claude Code

  .github/icon-dark.svg
  .github/icon-light.svg
  .github/workflows/style.yml
  README.md
  book/theme/favicon.png
  book/theme/favicon.svg

sync.sh --icon from TapHouse v6 (the shared configs are unchanged since v5),
and style.yml pins the drift check at v6 with icon: set, so these files are
guarded byte for byte from now on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JJQ9mTiYXTVqyTMKhoScnP
@tap
tap merged commit 393c666 into main Oct 8, 2026
36 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.

2 participants