Skip to content

feat: a favicon at last, the list of rules from betta design - #47

Merged
cbetta merged 1 commit into
masterfrom
favicons
Nov 8, 2026
Merged

cbetta merged 1 commit into
masterfrom
favicons

Conversation

@cbetta

@cbetta cbetta commented Nov 8, 2026

Copy link
Copy Markdown
Member
Reviewer check This PR
Auth / permissions No
Data access widened No
New secret, env var or key No
CI, deploy or infra No
Dependencies added or bumped No
Copilot Pending
Security review Not run: no gr4vy repo

Adds the favicons index.html has linked to since the archive, which were never committed, and clears them from scripts/known-broken-links.txt. The files come from betta design's hackcoc icon (betta-apps/betta-design#40).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cbetta
cbetta marked this pull request as ready for review November 8, 2026 17:47
@cbetta
cbetta requested a lite review from Copilot November 8, 2026 17:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues remain, and the favicon references, assets, and broken-links entries have been updated consistently.

No open issues 🎉

What changed in this PR

Adds the missing favicon assets for the archived site, updates public/index.html to reference local favicon URLs, and removes the resolved entries from the known-broken-links list.

Changes:

  • Adds local favicon assets from the betta design.
  • Replaces obsolete favicon references with local paths.
  • Removes resolved favicon URLs from the broken-links list.
File Summary
scripts/​known-broken-links.txt Removes favicon URLs that are no longer broken.
public/​index.html References the new local favicon assets.
public/​favicon.svg Adds the scalable favicon artwork with light and dark color support.

Review effort: Lite

@cbetta
cbetta merged commit ec6ce59 into master Nov 8, 2026
9 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