Skip to content

Conversation

@graphieros
Copy link
Contributor

  • Add github stars (if no stars, or not a GH repo, shows the 'repo' label)
  • Add forks count (if > 0)
  • Position npm link to top right

@vercel
Copy link

vercel bot commented Jan 25, 2026

@graphieros is attempting to deploy a commit to the danielroe Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 25, 2026

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

Project Deployment Review Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 25, 2026 5:47pm

Request Review

Copy link
Collaborator

@serhalp serhalp left a comment

Choose a reason for hiding this comment

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

LGTM, works great! Only thing I noticed is stars don't link to /stargazers.

>
<span class="i-carbon-logo-github w-4 h-4" aria-hidden="true" />
repo
<span v-if="stars"> {{ formatCompactNumber(stars, { decimals: 1 }) }} stars </span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: this should link to starsLink rather than to repositoryUrl

Copy link
Member

Choose a reason for hiding this comment

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

this replaces the previous repo link (in interests of saving space)

Copy link
Member

@danielroe danielroe left a comment

Choose a reason for hiding this comment

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

beautiful! ❤️

@danielroe danielroe merged commit 4013824 into npmx-dev:main Jan 25, 2026
6 checks passed
vinnymac pushed a commit to vinnymac/npmx.dev that referenced this pull request Jan 28, 2026
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.

3 participants