a
This commit is contained in:
parent
2bfed3e2ac
commit
2d48323604
1 changed files with 10 additions and 8 deletions
|
@ -4,21 +4,23 @@ PaperMC is happy you're willing to contribute to our projects. We are usually
|
|||
very lenient with all submitted PRs, but there are still some guidelines you
|
||||
can follow to make the approval process go more smoothly.
|
||||
|
||||
## Branch Protection
|
||||
## PR Origin
|
||||
|
||||
Paper will routinely modify your PR, whether it's a quick rebase or to take care
|
||||
of any minor nitpicks we might have. Often, it's better for us to solve these
|
||||
problems for you than make you go back and forth trying to fix them yourself.
|
||||
|
||||
Unfortunately, if you use an organization or the master branch for your PR,
|
||||
it prevents Paper from modifying it. This requires us to manually merge your
|
||||
PR, resulting in us closing the PR instead of marking it as merged.
|
||||
Unfortunately, if you use an organization for your PR, it prevents Paper from
|
||||
modifying it. See <https://github.com/isaacs/github/issues/1681> for more
|
||||
information on this issue.
|
||||
|
||||
We much prefer to have PRs show as merged, so please do not use repositories
|
||||
on organizations or the master branch for PRs.
|
||||
In addition, you also shouldn't open your PR from the master branch as this is
|
||||
considered bad practice and may cause problems for the Paper team or yourself.
|
||||
|
||||
> [!IMPORTANT]
|
||||
> Any PRs that do not follow these rules will be **AUTOMATICALLY CLOSED**, so
|
||||
> please do not use repositories on organizations or the master branch for PRs.
|
||||
|
||||
See <https://github.com/isaacs/github/issues/1681> for more information on the
|
||||
issue.
|
||||
|
||||
## Requirements
|
||||
|
||||
|
|
Loading…
Reference in a new issue