Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BUG Language Per Repo title does not match graph. #535

Open
alyashour opened this issue Nov 20, 2024 · 0 comments · May be fixed by #536
Open

BUG Language Per Repo title does not match graph. #535

alyashour opened this issue Nov 20, 2024 · 0 comments · May be fixed by #536
Assignees
@anmol098
Labels
bug Something isn't working

Comments

@alyashour
Copy link

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Run action on an account with repos owned by an organization in a language that outnumbers the top public repos language.
  2. The title will have a language that does not match the top lang.

For example: I have most of my repositories in Python (5). But as a student I have many repositories in JS (10) for assignments owned by my university.

This is what the code produces:

I Mostly Code in JavaScript

Python                   5 repos             ████░░░░░░░░░░░░░░░░░░░░░   14.29 % 
C++                      3 repos             ██░░░░░░░░░░░░░░░░░░░░░░░   08.57 % 
Swift                    2 repos             █░░░░░░░░░░░░░░░░░░░░░░░░   05.71 % 
HCL                      1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 % 
Go                       1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 %

Expected behavior
The code should not count org repositories in the title or start counting org repositories in the graph to be consistent.

Example:
I Mostly Code in Python

Python                   5 repos             ████░░░░░░░░░░░░░░░░░░░░░   14.29 % 
C++                      3 repos             ██░░░░░░░░░░░░░░░░░░░░░░░   08.57 % 
Swift                    2 repos             █░░░░░░░░░░░░░░░░░░░░░░░░   05.71 % 
HCL                      1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 % 
Go                       1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 %

Note that in this case the percentages are calculated incorrectly as it assumes all repositories as the total (35) instead of just owned repositories (19).

Correct Percentages
I Mostly Code in Python

Python                   5 repos             ████████░░░░░░░░░░░░░░░░░   26.32 % 
C++                      3 repos             ████░░░░░░░░░░░░░░░░░░░░░   15.79 % 
Swift                    2 repos             ███░░░░░░░░░░░░░░░░░░░░░░   10.53 % 
HCL                      1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   05.26 % 
Go                       1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   05.26 %

or

I Mostly Code in JavaScript

Javascript               10 repos            █████████████░░░░░░░░░░░░   45.45 % 
C#                       5 repos             ████░░░░░░░░░░░░░░░░░░░░░   14.29 % 
Python                   5 repos             ████░░░░░░░░░░░░░░░░░░░░░   14.29 % 
C++                      3 repos             ██░░░░░░░░░░░░░░░░░░░░░░░   08.57 % 
Swift                    2 repos             █░░░░░░░░░░░░░░░░░░░░░░░░   05.71 % 
HCL                      1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 % 
Go                       1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 % 
HTML                     1 repo              █░░░░░░░░░░░░░░░░░░░░░░░░   02.86 %

Github repository link
My Readme for reference. (https://github.com/alyashour)
@alyashour alyashour added the bug Something isn't working label Nov 20, 2024
@alyashour alyashour linked a pull request Nov 20, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anmol098 anmol098

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fixed top languages missing bug alyashour/waka-readme-stats
2 participants
@anmol098 @alyashour