Skip to content

feat(core): added gh workflow that adds 'contributor' label to PRs/Is…#8

Merged
goniz merged 1 commit intodevfrom
feat/gh-contributor-label
Jan 29, 2026
Merged

feat(core): added gh workflow that adds 'contributor' label to PRs/Is…#8
goniz merged 1 commit intodevfrom
feat/gh-contributor-label

Conversation

@goniz
Copy link
Owner

@goniz goniz commented Jan 29, 2026

…sues

What does this PR do?

How did you verify your code works?

@goniz goniz merged commit 884adae into dev Jan 29, 2026
1 check passed
@github-actions
Copy link

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5c13633eb5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +22 to +26
const isPR = !!context.payload.pull_request;
const issueNumber = isPR ? context.payload.pull_request.number : context.payload.issue.number;
const authorAssociation = isPR ? context.payload.pull_request.author_association : context.payload.issue.author_association;

if (authorAssociation === 'CONTRIBUTOR') {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Label misses first-time contributors

The workflow only adds the label when author_association is exactly 'CONTRIBUTOR', but GitHub uses values like FIRST_TIME_CONTRIBUTOR/FIRST_TIMER for first-time external contributors. That means new contributors’ PRs/issues won’t get the “contributor” label, which contradicts the stated intent to label contributor PRs/issues. Consider including the first-time values (or explicitly documenting that only repeat contributors are labeled).

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments