Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix contributors action creating multiple PRs #11693

Merged
merged 5 commits into from
Mar 6, 2025

Conversation

benelan
Copy link
Member

@benelan benelan commented Mar 6, 2025

Related Issue: #11690

Summary

The contributors action appends a uid to the branch name for some reason, which results in a new PR being created every time a commit lands on dev.

image

I forked the action and made a small change that lets us rely on a different action for the committing and PR creation/update process. We can switch back to the upstream action if my PR is accepted and installed.

ref: akhilmhdh/contributors-readme-action#84

@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Mar 6, 2025
@benelan benelan marked this pull request as draft March 6, 2025 12:21
@benelan benelan marked this pull request as ready for review March 6, 2025 12:55
@benelan benelan added the skip visual snapshots Pull requests that do not need visual regression testing. label Mar 6, 2025
@benelan benelan merged commit 07f163d into dev Mar 6, 2025
12 checks passed
@benelan benelan deleted the benelan/fix-multiple-contributor-prs branch March 6, 2025 17:16
benelan added a commit that referenced this pull request Mar 6, 2025
…m-contributors

* origin/dev:
  docs: update list of contributors (#11702)
  ci: fix contributors action creating multiple PRs (#11693)
  build(deps): update storybook monorepo to v8.6.3 (#11694)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants