Skip to content

feat: include gist stars into total stargazers#320

Open
qoomon wants to merge 1 commit into
ryo-ma:masterfrom
qoomon:patch-1
Open

feat: include gist stars into total stargazers#320
qoomon wants to merge 1 commit into
ryo-ma:masterfrom
qoomon:patch-1

Conversation

@qoomon

@qoomon qoomon commented Mar 21, 2025 •
edited
Loading

Copy link
Copy Markdown

fixes: #319

@vercel

vercel Bot commented Mar 21, 2025

Copy link
Copy Markdown

@qoomon is attempting to deploy a commit to the ryo-ma's Team Team on Vercel.

A member of the Team first needs to authorize it.

@qoomon qoomon mentioned this pull request Mar 21, 2025
Closed
@vercel

vercel Bot commented Mar 22, 2025 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
github-profile-trophy ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 22, 2025 0:08am

@ryo-ma

ryo-ma commented Mar 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Owner

It's a good idea, but there's the issue of the cost of Github Tokens.
As more data is retrieved by query, Github Tokens will run out.

@qoomon

qoomon commented Mar 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Author

@ryo-ma I had the same issue with my project https://github.com/qoomon/starlines My solution was to trigger github action workflow runs to gather the data from graphQL and cache them as github release artefacts. This works very well also for a bunch of results like querying 240k stars (incl. creating date) from https://github.com/facebook/react

Workflow

  • Call API Serverless Function
    • no cached image available
      • response with work in progress image
      • triggers GitHub Workflow
        • gather all (new) data
        • cache data as GitHub release artifact
        • create and cache image as GitHub release artifact
    • cached image available
      • response with cached image
      • trigger new workflow run in case cached image is outdated

RlxChap2
RlxChap2 reviewed Apr 2, 2025
View reviewed changes

@RlxChap2 RlxChap2 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.

actually this will help many people like me to add total stats ❤️👍

@pietrodicaprio pietrodicaprio mentioned this pull request May 4, 2025
Closed
@ryo-ma ryo-ma force-pushed the master branch 3 times, most recently from bd09792 to 08e0dcc Compare July 28, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@RlxChap2 RlxChap2 RlxChap2 left review comments

@jkowalleck jkowalleck jkowalleck approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature Request] Take GitHub Gist stars into account as well

4 participants

@qoomon @ryo-ma @jkowalleck @RlxChap2