Fix commits count when include_all_commits is true

[bokub]

The totalCommitsFetcher already sends the total number of commits, we don't need to add the values returned by the GraphQL API

Fixes #298

Current master:

My fork:

[codecov]

Codecov Report

Merging #391 into master will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #391      +/-   ##
==========================================
- Coverage   94.37%   94.35%   -0.03%     
==========================================
  Files          16       16              
  Lines         516      514       -2     
  Branches      140      140              
==========================================
- Hits          487      485       -2     
  Misses         25       25              
  Partials        4        4              

Impacted Files	Coverage Δ
src/fetchers/stats-fetcher.js	91.11% <100.00%> (-0.38%)	⬇️

---

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 9937aca...3f93846. Read the comment docs.

[bokub]

It's strange that the total commits returned by the experimental API are only 2.9k when there are 3.3k in the last year according to the GraphQL API....

Maybe the experimental API only counts public commits ? 🧐

[anuraghazra]

Maybe the experimental API only counts public commits ? monocle_face

Of course how would we even access the private commits.

[bokub]

Of course how would we even access the private commits.

You're right, but I wonder if you could access your private commits when deploying your own instance with your own PAT..

[bokub]

So is it OK to merge? Or do you want me to edit something?

[bokub]

Hello ?

[anuraghazra]

Hey @bokub really sorry for the late response, i totally forgot about this PR. this looks good to me! Thanks for the fix!

[bokub]

No problem, thanks!

[elfalehed]

Do you guys think that the API merge may have access to the private commits or what?

[techno-disaster]

The totalCommitsFetcher already sends the total number of commits, we don't need to add the values returned by the GraphQL API

Fixes #298

Current master:

My fork:

any idea why this shows more number of total commits in 2020 than total commits in all time?