Skip to content

ci(contrib): Exclude organization members from credits - #2271

Merged
jpnurmi merged 1 commit into
masterfrom
jpnurmi/ci/contrib-membership
Oct 29, 2026
Merged

jpnurmi merged 1 commit into
masterfrom
jpnurmi/ci/contrib-membership

Conversation

@jpnurmi

@jpnurmi jpnurmi commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

Check membership with the app token because author_association can classify organization members as contributors. Skip their credits and fail on lookup errors.

Follows the example of existing getsentry workflows:

Close: #2270

Check membership with the app token because `author_association` can
classify organization members as contributors. Skip their credits and
fail on lookup errors.

Follow existing workflows:
- `getsentry/sentry`: `.github/workflows/sentry-pull-request-bot.yml`
  checks membership with `github.rest.orgs.checkMembershipForUser`.
- `getsentry/sentry-docs`: `.github/workflows/codeowner_assignment.yaml`
  requests `permission-members: read` on its GitHub App token.

Close: #2270
@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.71%. Comparing base (83ee8d4) to head (bdd747f).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2271      +/-   ##
==========================================
+ Coverage   75.67%   75.71%   +0.03%     
==========================================
  Files         103      103              
  Lines       28319    28319              
  Branches     5210     5210              
==========================================
+ Hits        21431    21442      +11     
+ Misses       5550     5527      -23     
- Partials     1338     1350      +12     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpnurmi
jpnurmi merged commit 39ab761 into master Oct 29, 2026
70 checks passed
@jpnurmi
jpnurmi deleted the jpnurmi/ci/contrib-membership branch October 29, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants