Hide specific languages in "Top languages" card

[arjunmahishi]

Addressing #140

[codecov]

Codecov Report

Merging #150 into master will increase coverage by 0.21%.
The diff coverage is 97.05%.

@@            Coverage Diff             @@
##           master     #150      +/-   ##
==========================================
+ Coverage   94.47%   94.69%   +0.21%     
==========================================
  Files          15       15              
  Lines         362      377      +15     
  Branches      107      112       +5     
==========================================
+ Hits          342      357      +15     
  Misses         16       16              
  Partials        4        4              

Impacted Files	Coverage Δ
api/index.js	100.00% <ø> (ø)
api/top-langs.js	100.00% <ø> (ø)
src/fetchRepo.js	95.83% <ø> (ø)
src/retryer.js	73.68% <75.00%> (+1.46%)	⬆️
api/pin.js	95.23% <100.00%> (ø)
src/fetchStats.js	100.00% <100.00%> (ø)
src/fetchTopLanguages.js	100.00% <100.00%> (ø)
src/renderRepoCard.js	100.00% <100.00%> (ø)
src/renderTopLanguages.js	100.00% <100.00%> (ø)
src/utils.js	96.15% <100.00%> (+0.50%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03f55e8...1523091. Read the comment docs.

[martinRenou]

Shouldn't it be a change in the request instead?

Changing the request might recompute automatically the other percentages.

For example if I contributed to 60% C++ and 40% JavaScript, and wanted to hide the C++, I would like it to show that I contributed 100% JavaScript, not 40%.

[terror]

@martinRenou I'm pretty sure in order to recompute the percentage values, all we have to do is hide the languages before computing the total size

[anuraghazra]

Shouldn't it be a change in the request instead?

Changing the request might recompute automatically the other percentages.

For example if I contributed to 60% C++ and 40% JavaScript, and wanted to hide the C++, I would like it to show that I contributed 100% JavaScript, not 40%.

Yes the calculations should be relative to how many langs are shown

[anuraghazra]

LGTM Thanks @arjunmahishi

Note: I've deprecated hide_langs_below option.

[martinRenou]

Thanks for the amazing work, this repo deserves every single star it gets ;)