fix: fix JSON themes bug. #2544
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This fixes a JSON bug that was introduced in anuraghazra#2507.
|
@catelinemnemosyne is attempting to deploy a commit to the github readme stats Team on Vercel. A member of the Team first needs to authorize it. |
github-actions
bot
added
the
themes
rickstaa
approved these changes
Feb 25, 2023
Zo-Bro-23
approved these changes
Feb 25, 2023
pull bot
pushed a commit
to LucienZhang/github-readme-stats
that referenced
this pull request
Feb 25, 2023
This fixes a JSON bug that was introduced in anuraghazra#2507.
DPS0340
pushed a commit
to DPS0340/github-readme-stats
that referenced
this pull request
Mar 2, 2023
This fixes a JSON bug that was introduced in anuraghazra#2507.
dev-onejun
pushed a commit
to dev-onejun/github-readme-stats
that referenced
this pull request
Mar 3, 2023
This fixes a JSON bug that was introduced in anuraghazra#2507.
devantler
pushed a commit
to devantler/github-readme-stats
that referenced
this pull request
May 1, 2023
This fixes a JSON bug that was introduced in anuraghazra#2507.
necofuryai
added a commit
to necofuryai/github-readme-stats
that referenced
this pull request
May 19, 2025
This fixes a JSON bug that was introduced in anuraghazra#2507.
xhayper
pushed a commit
to xhayper/github-readme-stats
that referenced
this pull request
Jul 2, 2025
This fixes a JSON bug that was introduced in anuraghazra#2507.
Qreepex
added a commit
to Qreepex/github-readme-stats
that referenced
this pull request
Aug 7, 2025
* Include all repo w/ access to * parameterize affiliation * Use GraphQL variable syntax * Custom ownerAffiliations for general stats * Add default affiliation * Apply patches anuraghazra#1122 (comment) * build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373) Bumps [json5](https://github.com/json5/json5) from 2.2.1 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.2.1...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: update snapshots * Add loading Animation to Most used Language card (anuraghazra#2197) * add: text animation * compact progress done * cleaned code * non compact animation done * tests fixed + prettier * revert: vercel.json * remove: unnecessary import * added parameter in docs * style: improve syntax Co-authored-by: rickstaa <[email protected]> * ci: fix theme readme generation action (anuraghazra#2271) * Revert "build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373)" (anuraghazra#2393) This reverts commit 42fdff6. * Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394) This reverts commit 5ae959b. * Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396) This reverts commit 77dcdab. * fix: retryer fixed (anuraghazra#2400) * Revert "Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394)" (anuraghazra#2418) This reverts commit 70f0264. * refactor: update code formatting * Revert "Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396)" (anuraghazra#2419) This reverts commit 4b8198f. * Update readme.md (anuraghazra#2442) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data * feat: fix e2e tests * fix: fix e2e test data * feat: enable multi page star fetching for private vercel instances (anuraghazra#2159) * feat: enable multi-page stars' fetching for private vercel instances This commit enables multi-page stars' support from fetching on private Vercel instances. This feature can be disabled on the public Vercel instance by adding the `FETCH_SINGLE_PAGE_STARS=true` as an env variable in the public Vercel instance. This variable will not be present when people deploy their own Vercel instance, causing the code to fetch multiple star pages. * fix: improve stats multi-page fetching behavoir This commit makes sure that the GraphQL api is only called one time per 100 repositories. The old method added one unnecesairy GraphQL call. * docs: update documentation * style: improve code syntax Co-authored-by: Matteo Pierro <[email protected]> * lol happy new year * docs: remove rate limit documentation for now Remove the `FETCH_SINGLE_PAGE_STARS` from documentation for now since it might confuse people. * fix: fix error in automatic merge * feat: make sure env variable is read Co-authored-by: Matteo Pierro <[email protected]> Co-authored-by: Anurag <[email protected]> * chore: resolve conflict (anuraghazra#2453) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data Co-authored-by: rickstaa <[email protected]> * feat: parse role parameter * fix: change prod deployment branch to vercel branch to fix maxDuration bug (anuraghazra#2424) * Create deploy-prep.yml * Create deploy-prep.py * Update vercel.json * Update deploy-prep.yml * Update vercel.json * Added coauthor Co-authored-by: Dou Xiaobo <[email protected]> * Update deploy-prep.yml * refactor: format code * Added if condition to disable deployments on forks Co-authored-by: Rick Staa <[email protected]> * Update deploy-prep.yml Co-authored-by: Dou Xiaobo <[email protected]> Co-authored-by: Anurag Hazra <[email protected]> Co-authored-by: rickstaa <[email protected]> * fix: fixes card overflow problem anuraghazra#2452 (anuraghazra#2460) This commit makes sure that the card width is formatted correctly. * fix: fix language card ownerAffiliation bug This commit fixes a small error in the stats-fetcher that caused the `exclude_repo` parameter values to be seen as the `ownerAffiliation` inputs. * ci: prevent certain actions from running on forks (anuraghazra#2466) * feat: add PAT monitoring functions (anuraghazra#2178) * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * test: fix pat-info tests * Update api/status/pat-info.js Co-authored-by: Anurag Hazra <[email protected]> * test: fix broken tests * chore: fix suspended account * chore: simplify and refactor * chore: fix test * chore: add resetIn field --------- Co-authored-by: Anurag <[email protected]> * chore: minor changes in pat info (anuraghazra#2481) * ci: add update languages action (anuraghazra#2484) * ci: add update languages action * ci: make sure PR is created when upstream languages are updated * Add `rose` theme (anuraghazra#2480) Co-authored-by: 강민지 <[email protected]> * Support hide_progress for top-langs feature (anuraghazra#2514) * Add support for hide_progress in top languages feature * Fix mistake * Add documents for all languages * Remove unnecessary value check * Update top-languages-card.js * Revert document for all languages except English * Update documentation * Update documentation --------- Co-authored-by: Zohan Subhash <[email protected]> * add holi_theme (anuraghazra#2539) * add holi_theme * add holi_theme * Add one_dark_pro (anuraghazra#2507) This colors were taken from One Dark Pro theme in VSCode extention. Please add it and I will use it in my profile Co-authored-by: Zohan Subhash <[email protected]> * fix: fix JSON themes bug. (anuraghazra#2544) This fixes a JSON bug that was introduced in anuraghazra#2507. * Add auto-labelling for documentation updates (anuraghazra#2526) * ci: fix unsafe directory bug (anuraghazra#2518) This commit fixes a bug that was introduced due to a upstream change in the git package. See https://stackoverflow.com/questions/71849415/i-cannot-add-the-parent-directory-to-safe-directory-in-git/71904131#71904131 for more information. * Update readme.md (anuraghazra#2414) fix: missing "&" in show_icons=true in Showing icons section * test: update snapshots (anuraghazra#2519) * fix: for issue anuraghazra#2534 (anuraghazra#2536) * Add option to deploy using other services (anuraghazra#2525) * Create express.js * Update readme.md * Update readme.md * ci: update e2e tests (anuraghazra#2548) * ci: fix a bug in the theme preview action (anuraghazra#2549) * refactor: format code (anuraghazra#2550) * refactor: update languages JSON (anuraghazra#2554) Co-authored-by: rickstaa <[email protected]> * Preview theme workflow fix (anuraghazra#2557) * Fix octokit error * ci: make octokit instance global --------- Co-authored-by: rickstaa <[email protected]> * ci: preview theme workflow fix (anuraghazra#2559) * Fix octokit error * ci: make octokit instance global * Fix preview theme (move declarations to global) * refactor: make constants uppercase --------- Co-authored-by: rickstaa <[email protected]> * refactor: fix code comments and change 'up' rate limit (anuraghazra#2560) * Preview action fix (anuraghazra#2561) * Fix error * refactor: remove unused code --------- Co-authored-by: Rick Staa <[email protected]> * ci: fix theme preview action (anuraghazra#2563) * fix theme preview (anuraghazra#2564) * ci: fix theme preview action * fix: fix some bugs in the 'theme-preveiw' action * ci: fixes theme preview action (anuraghazra#2566) * ci: improve theme preview action (anuraghazra#2572) * Added "Shadow" set (Red, Green, Blue, transparent BG) (anuraghazra#2529) * Added "Shadow" set (Red, Green, Blue, transparent BG) 3 additional themes sticking primarily to flat colors, which the exception of icons and border being slightly darker. All 3 themes also have transparent backgrounds that will show differently per-user via GiHub's own light and dark themes. Transparency should also still provide easy readability for both. * Test Just want to see if we can make the themes have a transparent background. * Shadows moved under Transparent --------- Co-authored-by: Zohan Subhash <[email protected]> * ci: fix theme docs generate bug (anuraghazra#2573) * update my theme (anuraghazra#2576) * refactor: update languages JSON (anuraghazra#2596) Co-authored-by: rickstaa <[email protected]> * Add format stats option (anuraghazra#2155) * feat: added `format_stats` option (anuraghazra#2128) * refactor: change `format_stats` to `short_values` (anuraghazra#2128) * test: create shorten values test (anuraghazra#2128) * Update readme.md Co-authored-by: Rick Staa <[email protected]> * refactor: rename ``short_values`` to ``number_format`` * Update readme.md Co-authored-by: Rick Staa <[email protected]> * Update src/cards/stats-card.js Co-authored-by: Rick Staa <[email protected]> * refactor: format codebase --------- Co-authored-by: Rick Staa <[email protected]> * add github_dark_dimmed theme (anuraghazra#2594) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank_icon option (anuraghazra#2628) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <[email protected]> --------- Co-authored-by: Rick Staa <[email protected]> * Updated the custom error there was a Typo. (anuraghazra#2618) "Something went while trying to retrieve the stats data using the GraphQL API." -> "Something went wrong while trying to retrieve the stats data using the GraphQL API." * Fix typos inside wakatime test name (anuraghazra#2617) * removed redundant comparison (anuraghazra#957) * dev (anuraghazra#1782) * fix path to powered by vercel image (anuraghazra#1792) * Synchonize cache seconds min value inside docs translations with main readme (anuraghazra#2616) * New top language algorithm implementation (anuraghazra#1732) * Reduced vercel maxDuration * Implemented new algorithm for Top Langs * Revert "Reduced vercel maxDuration" This reverts commit b0bc626. * Added documentation * Fixed broken implementation * Update fetchTopLanguages.test.js Changed tests * Now uses the general formula The parameters p and q can be set by the user. * Updated tests and added new test * Added new test New test for order by repo count. * Updated documentation Added explanation and examples for new options. * Updated documentation This was overwritten in the merge commit. * docs: improve docs and fix tests * Renamed parameters Renamed `p` and `q` to `size_weight` and `count_weight`, respectively. * Updated the documentation Changes introduced in f2516d6 --------- Co-authored-by: rickstaa <[email protected]> * feat: improve CONTRIBUTING.md (anuraghazra#2609) * docs: update give logo (anuraghazra#2605) * Fixed typo in word color inside README (anuraghazra#2652) * Fixed docs typo inside src/common/utils.js (anuraghazra#2651) * Fixed several typos inside preview theme script (anuraghazra#2650) * Fixed todo inside stats card data fetcher (anuraghazra#2649) * docs: fix README language card usage order (anuraghazra#2659) * Fixed vscode type error in get card colors function (anuraghazra#2660) * Show no activity error in compact layout of wakatime card (anuraghazra#2662) * Fixed card constructor docstring to resolve vscode type errors (anuraghazra#2674) * Fixed wakatime create text node func docstring to resolve vscode type errors (anuraghazra#2678) * Fixed docstring for several wakatime card gen functions to resolve multiple type errors (anuraghazra#2690) * Fixed docstring for stats card create text node function to resolve type errors (anuraghazra#2689) * Update codecov action version to resolve github action deprecation warning (anuraghazra#2677) * Add PRs cache cleaning workflow (anuraghazra#2661) * build(deps): bump yaml from 2.1.3 to 2.2.2 (anuraghazra#2644) Bumps [yaml](https://github.com/eemeli/yaml) from 2.1.3 to 2.2.2. - [Release notes](https://github.com/eemeli/yaml/releases) - [Commits](eemeli/yaml@v2.1.3...v2.2.2) --- updated-dependencies: - dependency-name: yaml dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix(wakatime-fetcher): add default range to prevent user not found error (anuraghazra#2670) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <[email protected]> * add wakatime default range * update wakatime demos in readme with most active user * update wakatime demo in readme * remove &range=all_time from wakatime demos in readme --------- Co-authored-by: Rick Staa <[email protected]> * Cover with test changes in anuraghazra#2662 pull request (anuraghazra#2673) * test: fix mistaken pageInfo.endCursor keys (anuraghazra#2657) The previous mock logic was too simplistic and has been fixed in the commit. If the mock logic had been properly implemented, then the mistaken pageInfo.endCursor keys should have made the test case "should fetch two pages of stars if 'FETCH_MULTI_PAGE_STARS' env variable is set to `true`" stuck. * docs: update wakatime user (anuraghazra#2697) The old example user doesn't have a WakaTime account anymore. * Fixed docstring for get styles function to resolve vscode type errors (anuraghazra#2700) * Fixed axios imports inside docstrings to resolve vscode type errors (anuraghazra#2699) * Fixed typos inside bug report issue template (anuraghazra#2706) * Update wakatime user inside docs translations (anuraghazra#2705) * Fixed docstring for create language text node function (anuraghazra#2704) * Fixed docstring for wakatime card recalculate percentages function (anuraghazra#2710) * add pie chart layout to language card (anuraghazra#2099) * add pie chart layout to language card * resolve failing top-lang card tests * scale down pie chart * update readme.md * Update readme.md Co-authored-by: Rick Staa <[email protected]> * style: format code * update donut layout to be created without dependencies * minor update * style: format readme * resolve failing tests * refactor: clean up code and add extra tests This commit cleans up the pie chart generation code and adds additional tests. * feat: improve pie chart positioning * rename layout pie to donut * add animation to donut layout * refactor: rename pie and doughnut to donut * feat: decrease donus animation delay --------- Co-authored-by: rickstaa <[email protected]> * Fixed docstring for create donut language node function (anuraghazra#2713) * Add missing argument to create compact lang node function to resolve vscode type error (anuraghazra#2714) * Top languages card pie layout (anuraghazra#2709) * Top languages card donut layout * Top languages card pie layout * renames * dev * docs * dev * dev * animations * dev * handle one language * Docs: add missing top languages card layouts. (anuraghazra#2717) * Top languages card render test: move repeated code into helper function (anuraghazra#2718) * Docs (translations): add missing top languages card layouts. (anuraghazra#2728) * Top languages card donut vertical layout (anuraghazra#2701) * Top languages card donut layout * dev * dev * dev * dev * Docs (translations): add top languages card donut vertical layout (anuraghazra#2739) * Docs: fix top languages card available layouts number (anuraghazra#2738) * Ranking System v2 (anuraghazra#1186) * Revise rank calculation * Replace contributions by commits * Lower average stats and S+ threshold * Fix calculateRank.test.js Missing key in dictionary constructor Co-authored-by: Rick Staa <[email protected]> * refactor: run prettier * feat: change star weight to 0.75 * Separate PRs and issues * Tweak weights * Add count_private back * fix: enable 'count_private' again * test: fix tests * refactor: improve code formatting * Higher targets --------- Co-authored-by: Rick Staa <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Raymond Nook <[email protected]> Co-authored-by: rickstaa <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Huzaifa Khilawala <[email protected]> Co-authored-by: Anurag Hazra <[email protected]> Co-authored-by: Zohan Subhash <[email protected]> Co-authored-by: Matteo Pierro <[email protected]> Co-authored-by: Dou Xiaobo <[email protected]> Co-authored-by: 강민지 <[email protected]> Co-authored-by: 강민지 <[email protected]> Co-authored-by: Amir <[email protected]> Co-authored-by: Mohamed Hassan <[email protected]> Co-authored-by: Oleksandr Perlov <[email protected]> Co-authored-by: Cateline Mnemosyne <[email protected]> Co-authored-by: Raphael Gonçalves <[email protected]> Co-authored-by: Rehman <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: rickstaa <[email protected]> Co-authored-by: Etanarvazac Revorix <[email protected]> Co-authored-by: Eduardo Zaniboni <[email protected]> Co-authored-by: Caeden Perelli-Harris <[email protected]> Co-authored-by: Fabiano Couto <[email protected]> Co-authored-by: LOKESH SINGH <[email protected]> Co-authored-by: Alexandr Garbuzov <[email protected]> Co-authored-by: kitswas <[email protected]> Co-authored-by: Rongrong <[email protected]> Co-authored-by: Nabil Alamin <[email protected]> Co-authored-by: Alexandr Garbuzov <[email protected]> Co-authored-by: François Rozet <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes a JSON bug that was introduced in #2507.