Skip to content

chore: enforce no merge commits in PRs - #10576

Merged
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:carson-no-merge-commits
Sep 22, 2026
Merged

paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:carson-no-merge-commits

Conversation

@paulbalandan

@paulbalandan paulbalandan commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Description
This forbids merge commits when updating PRs, which is already written in

> Do not merge the upstream target branch into your contribution branch. A merge

Exemptions are the release PRs: (1) develop to master, and (2) 4.x to develop.

Offending PRs get a failing check run.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@michalsn michalsn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@paulbalandan
paulbalandan merged commit 81af068 into codeigniter4:develop Sep 22, 2026
7 checks passed
@paulbalandan
paulbalandan deleted the carson-no-merge-commits branch September 22, 2026 04:22
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.

3 participants