Customization of ownerAffiliations with parameter

[developStorm]

See #1 (comment). #1

[vercel]

@developStorm is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

[rickstaa]

@developStorm Thanks a lot for this pull request it works perfectly on my fork.

[tyt2y3]

Thank you so much! I find this useful 😀
I look forward to this being merged.

[rickstaa]

What is blocking this from being merged?

[rickstaa]

@developStorm I reviewed your pull request and it looks good to me. I however noticed that the tests were broken. I created a pull request to fix this on your branch developStorm#1.

@anuraghazra Sorry for the PR waterfall. I, however, found too many changes for me to use the review suggestion tag.

[rickstaa]

I think this solution is more in line with #1 (comment).

[rickstaa]

Apart from these changes, I think the new argument should be added to the documentation before this pull request can be merged.

[anuraghazra]

Automated Theme Preview

• ⚠️ title_color does not passes AA contrast ratio
• ⚠️ text_color does not passes AA contrast ratio

title_color: #b57614 | icon_color: #af3a03 | text_color: #427b58 | bg_color: #fbf1c7

Preview Link

Hi, thanks for the theme contribution, please read our theme contribution guidelines.

We are currently only accepting color combinations from any VSCode theme or themes which have good color combination to minimize bloating the themes collection.

Also note that if this theme is exclusively for your personal use, then instead of adding it to our theme collection you can use card customization options

[rickstaa]

Great, everything seems to work now. Thanks!

[codecov]

Codecov Report

Attention: Patch coverage is 93.15068% with 5 lines in your changes missing coverage. Please review.

Project coverage is 96.84%. Comparing base (4b17300) to head (7337505).
Report is 675 commits behind head on master.

Files with missing lines	Patch %	Lines
src/common/utils.js	86.84%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1122      +/-   ##
==========================================
- Coverage   96.90%   96.84%   -0.07%     
==========================================
  Files          22       22              
  Lines        3882     3956      +74     
  Branches      337      341       +4     
==========================================
+ Hits         3762     3831      +69     
- Misses        118      123       +5     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rickstaa]

@developStorm I created #2277 as a replacement for #1122. Please take a look and close this PR if you think it suits your needs.

[rickstaa]

@anuraghazra I'm okay with merging this since it is what the community wants. As far as I know, it will not deplete our PATs since it uses the same number of GraphQL points as the master branch. It will, however, not give accurate stats results on the Public instance unless FETCH_MULTI_PAGE_STARS is enabled. It also will make the language results more incorrect since we currently get the language results from the repository object (see #1122 (comment)). Maybe let's release this feature under the INCLUDE_ORGS environment variable? We can also remove the option entirely in the language card if you don't like the inaccuracies. With any option we still need to add:

• Tests for the new role variable.
• Merge fix: Fetch all repos for top languages #2111 under a MULTI_PAGE_LANGUAGES env variable so that it can be used in the GitHub action.
• Maybe some documentation explaining the role variable and the language and stats card inaccuracies. We can add a section on experimental features in the readme.md.

[rickstaa]

Replaced by #2459 since I don't want to break @developStorm's master branch when I make a change.