Skip to content

Comments

feat: include gist stars into total stargazers#320

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

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

Conversation

@qoomon
Copy link

@qoomon qoomon commented Mar 21, 2025 •
edited
Loading

fixes: #319

@vercel
Copy link

vercel bot commented Mar 21, 2025

@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
Copy link

vercel bot commented Mar 22, 2025 •
edited
Loading

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
Copy link
Owner

ryo-ma commented Mar 25, 2025 •
edited
Loading

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
Copy link
Author

qoomon commented Mar 25, 2025 •
edited
Loading

@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
Copy link

@RlxChap2 RlxChap2 left a comment

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