Skip to content

docs: clarify contribution responsibilities and PR limits - #10770

Merged
michalsn merged 2 commits into
codeigniter4:developfrom
michalsn:docs/contriguting
Nov 12, 2026
Merged

michalsn merged 2 commits into
codeigniter4:developfrom
michalsn:docs/contriguting

Conversation

@michalsn

Copy link
Copy Markdown
Member

Description
This PR clarifies contributor responsibilities when using AI tools and limits contributors without write access to three open PRs, including drafts, with exceptions for trusted contributors.

The limit will be enforced via a GitHub repository setting. I'm open to discussion about whether three is a good number or whether we should start with a limit of two PRs (or other number).

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

@paulbalandan paulbalandan 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.

Can we add a section in the AGENTS.md, maybe as the first section, reminding agents not to open a PR unless the requirements are met and tell the user as such.

@michalsn

Copy link
Copy Markdown
Member Author

Done, but I'm not sure if this is the best approach. Feel free to suggest changes.

@paulbalandan paulbalandan 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.

Looks good.

@michalsn
michalsn merged commit c431aa1 into codeigniter4:develop Nov 12, 2026
7 checks passed
@michalsn

Copy link
Copy Markdown
Member Author

A limit of 3 open PRs per contributor is now in place for this repository.

@neznaika0

Copy link
Copy Markdown
Contributor

What will the situation look like when a user has a long-open PR? Close them?

@michalsn

Copy link
Copy Markdown
Member Author

What will the situation look like when a user has a long-open PR? Close them?

Pretty much, yes. Usually, there's a good reason why a PR stays open for a long time. And with a limit of 3 open PRs per contributor, there's still room to submit other contributions in the meantime.

@michalsn
michalsn deleted the docs/contriguting branch November 12, 2026 10:23
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.

4 participants