Skip to content

chore: accept pull requests only from branches of this repository - #1

Merged
aditeyabaral merged 2 commits into
mainfrom
chore/same-repo-branches
Nov 13, 2026
Merged

aditeyabaral merged 2 commits into
mainfrom
chore/same-repo-branches

Conversation

@aditeyabaral

@aditeyabaral aditeyabaral commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

pesu-dev is moving from fork pull requests to branches of each repository: contributors clone the repository itself, push a branch that is not main, and open the pull request from it. Forks are not accepted (see pesu-dev/skills#10 and pesu-dev/auth#267).

This builds on @Vector316I71A's work in pesu-dev/auth#266 and pesu-dev/skills#9, which started the move. Thank you!

File Change
.github/workflows/ci.yml New Validate PR origin job: fails a PR from a fork (pointing at the README for write access), from main, or to anything but main. It runs on pull requests only
README.md The planned shared PR Source Checker now describes the same-repository rule; a short Contributing section (clone, branch, PR to main, maintainers tag releases); the structure table mentions the new check

Nothing here changes the reusable workflows, so no release or v1 move is needed.

After merging, a maintainer should protect main:

  • require a PR, a review and the Validate PR origin check;
  • restrict v* tags, v1 included, to maintainers.

developer-team can push here, and pesu-dev/auth runs deploy_render.yml@v1 with its production secrets.

Checked with actionlint.

🤖 Generated with Claude Code

pesu-dev contributors now clone the repository itself, push a branch that is not main, and open the
pull request from it; forks are not accepted.

CI gets a Validate PR origin job that fails a pull request from a fork, from main, or to anything
but main, so the rule can be required on main like the other repositories' origin checks. The
README's planned shared PR Source Checker now describes the same rule, and a Contributing section
says how to contribute and that maintainers tag releases.

Checked with actionlint.
The PR origin check told a fork's author to push to this repository, which needs write access they
usually do not have yet. Point them at the README's Contributing section.
@aditeyabaral
aditeyabaral requested a review from sach-12 November 13, 2026 05:36
@aditeyabaral
aditeyabaral merged commit fec5232 into main Nov 13, 2026
3 checks passed
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.

1 participant