Merge pull request #13 from plone/gh-contributions-merged-prs-only

Contributor Stats: count only merged PRs

Author: tisto

README.md

@@ -249,3 +249,24 @@ make run-organisation-stats-2022
 - **Without token**: 60 requests per hour (not recommended)
 - Built-in rate limiting and retry logic
 - Handles pagination for large datasets
+
+## Data Collection Methodology
+
+### Pull Request Counting
+
+All contributor statistics count only **merged** pull requests.
+The counts exclude the following pull request statuses.
+- Open pull requests (still under review)
+- Closed but not merged pull requests (rejected)
+
+This ensures statistics reflect actual contributions to the codebase.
+
+**Exception**: PR interaction statistics (`plone_pr_interactions.py`) count all interactions (comments, reviews) on any PR, regardless of merge status, to accurately capture community engagement.
+
+### Commit Counting
+
+Commits are counted from the default branch only, which automatically includes:
+- Direct commits to the default branch
+- Commits from merged pull requests
+
+This excludes commits on unmerged branches or rejected pull requests.

data/2015-plone-contributors.csv

@@ -1,8 +1,8 @@
 github_username,pull_requests,commits
 robgietema,11,164
 cekk,2,2
-VaysseB,2,7
-jaroel,2,6
+VaysseB,1,7
+jaroel,1,6
 thet,1,1
 tareqalam,1,1
 tisto,0,3

data/2016-plone-contributors.csv

@@ -1,16 +1,14 @@
 github_username,pull_requests,commits
-robgietema,51,278
-sneridagh,36,186
-tisto,27,168
-nileshgulia1,25,11
-bloodbare,9,61
+robgietema,49,278
+sneridagh,35,186
+tisto,22,168
+nileshgulia1,16,11
+bloodbare,6,61
 davisagli,4,4
-cekk,3,18
-pnicolli,2,1
-vangheem,1,5
+cekk,2,18
+pnicolli,1,1
 ajayns,1,9
 esteele,1,0
 pbauer,1,1
-mikejmets,1,0
-thet,1,0
 datakurre,0,18
+vangheem,0,5