Skip to content

fix: for issue #2534 #2536

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 25, 2023

Conversation

md-rehman
Copy link
Contributor

Updated the code and respective test case as well. And also fixed for minimum width difference as well. Here is the preview

Without the Fix

Screenshot 2023-02-22 at 5 12 15 PM

With the Fix

Screenshot 2023-02-22 at 5 11 53 PM

Increased Min width to accommodate for reduced width

Screenshot 2023-02-22 at 5 38 23 PM

@vercel
Copy link

vercel bot commented Feb 22, 2023

@md-rehman is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@Zo-Bro-23 Zo-Bro-23 left a comment

Choose a reason for hiding this comment

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

Good for merge 👍

@Zo-Bro-23 Zo-Bro-23 requested a review from rickstaa February 25, 2023 12:01
@codecov
Copy link

codecov bot commented Feb 25, 2023

Codecov Report

Base: 97.27% // Head: 97.27% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (c6e00ec) compared to base (ba7c2f8).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2536      +/-   ##
==========================================
- Coverage   97.27%   97.27%   -0.01%     
==========================================
  Files          24       24              
  Lines        4178     4177       -1     
  Branches      384      384              
==========================================
- Hits         4064     4063       -1     
  Misses        112      112              
  Partials        2        2              
Impacted Files Coverage Δ
src/cards/top-languages-card.js 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rickstaa rickstaa merged commit 2ab8b85 into anuraghazra:master Feb 25, 2023
DPS0340 pushed a commit to DPS0340/github-readme-stats that referenced this pull request Mar 2, 2023
dev-onejun pushed a commit to dev-onejun/github-readme-stats that referenced this pull request Mar 3, 2023
devantler pushed a commit to devantler/github-readme-stats that referenced this pull request May 1, 2023
necofuryai added a commit to necofuryai/github-readme-stats that referenced this pull request May 19, 2025
xhayper pushed a commit to xhayper/github-readme-stats that referenced this pull request Jul 2, 2025
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] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
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 <rick.staa@outlook.com>

* 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 <pierromatteo@gmail.com>

* 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 <pierromatteo@gmail.com>
Co-authored-by: Anurag <hazru.anurag@gmail.com>

* 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 <rick.staa@outlook.com>

* 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 <93511091+douxiaobo@users.noreply.github.com>

* Update deploy-prep.yml

* refactor: format code

* Added if condition to disable deployments on forks

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* Update deploy-prep.yml

Co-authored-by: Dou Xiaobo <93511091+douxiaobo@users.noreply.github.com>
Co-authored-by: Anurag Hazra <hazru.anurag@gmail.com>
Co-authored-by: rickstaa <rick.staa@outlook.com>

* 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 <hazru.anurag@gmail.com>

* test: fix broken tests

* chore: fix suspended account

* chore: simplify and refactor

* chore: fix test

* chore: add resetIn field

---------

Co-authored-by: Anurag <hazru.anurag@gmail.com>

* 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: 강민지 <68285922+Kminzzi@users.noreply.github.com>

* 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 <zohan.subhash@gmail.com>

* 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 <zohan.subhash@gmail.com>

* 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 <rickstaa@users.noreply.github.com>

* Preview theme workflow fix (anuraghazra#2557)

* Fix octokit error

* ci: make octokit instance global

---------

Co-authored-by: rickstaa <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <zohan.subhash@gmail.com>

* ci: fix theme docs generate bug (anuraghazra#2573)

* update my theme (anuraghazra#2576)

* refactor: update languages JSON (anuraghazra#2596)

Co-authored-by: rickstaa <rickstaa@users.noreply.github.com>

* 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 <rick.staa@outlook.com>

* refactor: rename ``short_values`` to ``number_format``

* Update readme.md

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* Update src/cards/stats-card.js

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* refactor: format codebase

---------

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

---------

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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] <support@github.com>
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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

* 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 <rick.staa@outlook.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Raymond Nook <59678453+developStorm@users.noreply.github.com>
Co-authored-by: rickstaa <rick.staa@outlook.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Huzaifa Khilawala <55500003+RedHeadphone@users.noreply.github.com>
Co-authored-by: Anurag Hazra <hazru.anurag@gmail.com>
Co-authored-by: Zohan Subhash <zohan.subhash@gmail.com>
Co-authored-by: Matteo Pierro <pierromatteo@gmail.com>
Co-authored-by: Dou Xiaobo <93511091+douxiaobo@users.noreply.github.com>
Co-authored-by: 강민지 <68285922+Meezzi@users.noreply.github.com>
Co-authored-by: 강민지 <68285922+Kminzzi@users.noreply.github.com>
Co-authored-by: Amir <amirhhakimnejad@gmail.com>
Co-authored-by: Mohamed Hassan <mo.hassan2792@gmail.com>
Co-authored-by: Oleksandr Perlov <oleksandrperlov@gmail.com>
Co-authored-by: Cateline Mnemosyne <123184375+catelinemnemosyne@users.noreply.github.com>
Co-authored-by: Raphael Gonçalves <89359384+raphaelricardo10@users.noreply.github.com>
Co-authored-by: Rehman <rehman.2468@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: rickstaa <rickstaa@users.noreply.github.com>
Co-authored-by: Etanarvazac Revorix <etanarvazac@outlook.com>
Co-authored-by: Eduardo Zaniboni <67515606+eduardozaniboni@users.noreply.github.com>
Co-authored-by: Caeden Perelli-Harris <caedenperelliharris@gmail.com>
Co-authored-by: Fabiano Couto <facodeli@gmail.com>
Co-authored-by: LOKESH SINGH <77314004+LokeshXs@users.noreply.github.com>
Co-authored-by: Alexandr Garbuzov <53787217+qwerty541@users.noreply.github.com>
Co-authored-by: kitswas <90329875+kitswas@users.noreply.github.com>
Co-authored-by: Rongrong <i@rong.moe>
Co-authored-by: Nabil Alamin <aminonimisi2@gmail.com>
Co-authored-by: Alexandr Garbuzov <qwerty541zxc@gmail.com>
Co-authored-by: François Rozet <francois.rozet@outlook.com>
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