Skip to content

Commit 3cbbb0b

Browse files
authored
Fix typos & missing commas in CONTRIBUTING.rst
1 parent fbe6624 commit 3cbbb0b

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

CONTRIBUTING.rst

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,7 @@ Who does the backporting
409409
~~~~~~~~~~~~~~~~~~~~~~~~
410410

411411
As mentioned above, bugs should first be fixed on ``main`` (except in rare occasions
412-
that a bug only happens in a previous release). So who should do the backport procedure described
412+
that a bug only happens in a previous release). So, who should do the backport procedure described
413413
above?
414414

415415
1. If the bug was fixed by a core developer, it is the main responsibility of that core developer
@@ -447,7 +447,7 @@ can always reopen the issue/pull request in their own time later if it makes sen
447447
When to close
448448
~~~~~~~~~~~~~
449449

450-
Here are a few general rules the maintainers use to decide when to close issues/PRs because
450+
Here are a few general rules the maintainers use deciding when to close issues/PRs because
451451
of lack of inactivity:
452452

453453
* Issues labeled ``question`` or ``needs information``: closed after 14 days inactive.
@@ -459,15 +459,15 @@ The above are **not hard rules**, but merely **guidelines**, and can be (and oft
459459
Closing pull requests
460460
~~~~~~~~~~~~~~~~~~~~~
461461

462-
When closing a Pull Request, it needs to be acknowledge the time, effort, and interest demonstrated by the person which submitted it. As mentioned previously, it is not the intent of the team to dismiss stalled pull request entirely but to merely to clear up our queue, so a message like the one below is warranted when closing a pull request that went stale:
462+
When closing a Pull Request, it needs to be acknowledging the time, effort, and interest demonstrated by the person which submitted it. As mentioned previously, it is not the intent of the team to dismiss a stalled pull request entirely but to merely to clear up our queue, so a message like the one below is warranted when closing a pull request that went stale:
463463

464464
Hi <contributor>,
465465

466-
First of all we would like to thank you for your time and effort on working on this, the pytest team deeply appreciates it.
466+
First of all, we would like to thank you for your time and effort on working on this, the pytest team deeply appreciates it.
467467

468468
We noticed it has been awhile since you have updated this PR, however. pytest is a high activity project, with many issues/PRs being opened daily, so it is hard for us maintainers to track which PRs are ready for merging, for review, or need more attention.
469469

470-
So for those reasons we think it is best to close the PR for now, but with the only intention to cleanup our queue, it is by no means a rejection of your changes. We still encourage you to re-open this PR (it is just a click of a button away) when you are ready to get back to it.
470+
So for those reasons we, think it is best to close the PR for now, but with the only intention to clean up our queue, it is by no means a rejection of your changes. We still encourage you to re-open this PR (it is just a click of a button away) when you are ready to get back to it.
471471

472472
Again we appreciate your time for working on this, and hope you might get back to this at a later time!
473473

0 commit comments

Comments
 (0)