Skip to content

feat: add option to skip internal contributors#94

Merged
plbstl merged 24 commits into
mainfrom
skip-internal-contributors
Mar 29, 2026
Merged

feat: add option to skip internal contributors#94
plbstl merged 24 commits into
mainfrom
skip-internal-contributors

Conversation

@plbstl

@plbstl plbstl commented Mar 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Internal contributors are organization members and repository collaborators.

To enable this behavior, set skip-internal-contributors to true, and populate the GH_PAT_READ_ORG env variable.

steps:
  - uses: plbstl/first-contribution@commit-sha
    env:
      GH_PAT_READ_ORG: ${{ secrets.GH_PAT_READ_ORG }}
    with:
      skip-internal-contributors: true
      # other options...

Create a Personal Access Token (PAT) with https://github.com/settings/tokens/new?scopes=read:org. Only read:org scope is needed.

Closes #93

plbstl added 21 commits March 28, 2026 17:15
@plbstl
add skip-internal-contributors option to action.yml and README
09d3cd3
@plbstl
implement skip-internal-contributors option
47a692e
@plbstl
log author_association when exiting
a5060fa
@plbstl
regenerate dist
70cce9a
@plbstl
update jscpd report
83985f0
@plbstl
Merge branch 'main' of https://github.com/plbstl/first-contribution i…
60891fb 
…nto skip-internal-contributors
@plbstl
always debug print out author-association
4e20bb3
@plbstl
regenerate coverage & jscpd reports
1c3ccd1
@plbstl
relocate author association debug message
2122827
@plbstl
reword debug logs for internal contributor
8b95742
@plbstl
add 'CONTRIBUTOR' author association as an internal contributor
ddaf102
@plbstl
use github api to check for internal collaborators
428d381 
author association so brittle, part of why i didn't use it for other stuff in the first place
@plbstl
remove author_association usage from tests
dc60a11
@plbstl
update tests for checking internal contributors
722b7cf
@plbstl
ensure 'Consider enabling skip-internal-contributors...' logs corre…
5b2ae84 
…ctly

it was impossible to execute `if (skip_internal_contributors_is_false) { core.info...`
@plbstl
fix checkCollaborator mock values in main.test.ts
c24ecaa
@plbstl
fix typescript errors
fd2944f
@plbstl
generate reports
3e9903f
@plbstl
fix is_internal_contributor error handling
5dd53da 
repos.checkCollaborator throws an error if the user is not a collaborator
@plbstl
make typescript happy
b9fdcc9
@plbstl
generate reports
b414024
@plbstl plbstl force-pushed the skip-internal-contributors branch from 421e2dc to 1067847 Compare March 29, 2026 16:46
@plbstl
fix org member check
488122d 
Users will need a custom PAT for 'read:org'
@plbstl plbstl force-pushed the skip-internal-contributors branch from 1067847 to 488122d Compare March 29, 2026 19:08
@plbstl plbstl marked this pull request as ready for review March 29, 2026 19:32
@plbstl plbstl merged commit ad90c0b into main Mar 29, 2026
31 checks passed
@plbstl plbstl deleted the skip-internal-contributors branch March 29, 2026 19:43
@plbstl plbstl added the enhancement New feature or request label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add an optional property to ignore organisation members

1 participant

@plbstl