Skip to content

chore(icon): Use official Hugging Face logo and update cache data #51

Merged
FenjuFu merged 1 commit into
mainfrom
change-hf-icon
Jul 9, 2026
Merged

chore(icon): Use official Hugging Face logo and update cache data #51
FenjuFu merged 1 commit into
mainfrom
change-hf-icon

Conversation

@dongjiang1989

Copy link
Copy Markdown
Contributor

Summary

  • Replace inline placeholder Hugging Face SVG with the official logo downloaded from ollama.com, stored locally at src/assets/images/hugging-face.svg
  • Bump version to 0.3.0

Changes

Added

  • src/assets/images/hugging-face.svg — official Hugging Face SVG logo asset

Changed

  • ProjectCard.astro — replace inline SVG with <img src={hfIcon.src}>
  • [slug].astro (project detail) — replace inline SVG with <img src={hfIcon.src}>
  • .cache/project-stats.json and .cache/contributors.json — refreshed
  • package.json / package-lock.json — version 0.2.0 → 0.3.0
  • CHANGELOG.md — add Hugging Face icon asset and cache refresh to [Unreleased]

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Use official Hugging Face logo and update cache data

Signed-off-by: dongjiang <dongjiang1989@126.com>
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Jul 8, 2026 7:58am

@FenjuFu FenjuFu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Clean swap from placeholder SVG to the official Hugging Face logo, changes are well-scoped and all CI is green. Minor nit (non-blocking): the img uses h-4/w-4 (square) while the source SVG is 32×30 — a ~6% width squish, imperceptible at icon size.

@FenjuFu FenjuFu merged commit d6553b1 into main Jul 9, 2026
12 checks passed
@FenjuFu FenjuFu deleted the change-hf-icon branch July 9, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants