github-readme-stats icon indicating copy to clipboard operation
github-readme-stats copied to clipboard

feat: add `include_orgs` card variable

Open rickstaa opened this issue 3 years ago • 4 comments

This pull request adds experimental Organization support. Users can include stats from Organization repositories to which they are collaborators using the include_orgs card argument. Please be aware that because of #1852, only the first 100 repositories are used. Including organization, stats might therefore skew your results.

TODOs

  • [ ] Add a test that checks whether Axios receives the right request query. I tried to google how this is done but could not yet find a good solution. @anuraghazra, do you know how we can implement such a test?

rickstaa avatar Nov 21 '22 11:11 rickstaa

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
github-readme-stats ✅ Ready (Inspect) Visit Preview Nov 22, 2022 at 8:51AM (UTC)

vercel[bot] avatar Nov 21 '22 11:11 vercel[bot]

Codecov Report

Base: 96.79% // Head: 96.81% // Increases project coverage by +0.02% :tada:

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2277      +/-   ##
==========================================
+ Coverage   96.79%   96.81%   +0.02%     
==========================================
  Files          22       22              
  Lines        3835     3862      +27     
  Branches      328      331       +3     
==========================================
+ Hits         3712     3739      +27     
  Misses        121      121              
  Partials        2        2              
Impacted Files Coverage Δ
api/index.js 96.03% <100.00%> (+0.07%) :arrow_up:
api/top-langs.js 95.34% <100.00%> (+0.11%) :arrow_up:
src/common/utils.js 100.00% <100.00%> (ø)
src/fetchers/stats-fetcher.js 91.46% <100.00%> (+0.46%) :arrow_up:
src/fetchers/top-languages-fetcher.js 87.75% <100.00%> (+0.70%) :arrow_up:

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

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

codecov[bot] avatar Nov 21 '22 11:11 codecov[bot]

@anuraghazra Are there any problems with merging this PRs other than the results being skewed because of https://github.com/anuraghazra/github-readme-stats/issues/1852?

rickstaa avatar Nov 21 '22 11:11 rickstaa

@anuraghazra Maybe we should add a note about https://github.com/anuraghazra/github-readme-stats/issues/1852 in the readme when merging this PR.

rickstaa avatar Nov 21 '22 11:11 rickstaa

Replaced by #2459 since I don't mind giving people more control.

rickstaa avatar Jan 24 '23 13:01 rickstaa