Skip to content

Carry the Tap family icon: README title; drift check v6 - #39

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

PythonTap's mark from TapHouse brand/ now appears in the ReadMe.md title. The files are .github/icon-light.svg and icon-dark.svg, switched by the reader's GitHub theme. style.yml pins the drift check at TapHouse v6 with icon: PythonTap, which also guards the package's icon.png. That file is already TapHouse's PythonTap-package.png byte for byte, so this PR leaves it unchanged.

Why

This is the family icon rollout (tap/TapHouse#10, #11). The files come from sync.sh --icon PythonTap 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, icon.png included.
  • 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 changes, and nothing in the repo links to it.

Notes for the reviewer

  • The ReadMe ships in the package (assemble-package.py copies it), but .github/ does not. A reader opening the package's ReadMe.md in a Markdown viewer sees no mark in the title; alt="", so nothing else shows either. Max does not render the ReadMe. I left the package contents alone. Shipping the two SVGs would be a change to assemble-package.py if you want the mark there too.

🤖 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

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 <[email protected]>
Claude-Session: https://claude.ai/code/session_01JJQ9mTiYXTVqyTMKhoScnP
@tap
tap merged commit 87b0f15 into main Oct 8, 2026
18 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