Opened 2 hours ago
Last modified 46 minutes ago
#35894 assigned Cleanup/optimization
Rename "patch" to "pull request" in the contributor documentation
| Reported by: | Baptiste Mispelon | Owned by: | Baptiste Mispelon |
|---|---|---|---|
| Component: | Documentation | Version: | 5.0 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
No one effectively contributes via patches anymore (according to the Trac database, only ~10 patches were submitted in the past 12 months), I think it's time we update Trac and our documentation to reflect that.
I've created a PR to change the wording on Trac itself: https://github.com/django/code.djangoproject.com/pull/224
What's left now is to update our contributing guides in the documentation.
Attachments (1)
Change History (3)
by , 69 minutes ago
| Attachment: | issue35894.diff added |
|---|
comment:1 by , 68 minutes ago
| Has patch: | set |
|---|
comment:2 by , 46 minutes ago
I also considered replacing the expression "ready for checkin" with "ready for merging" (which has the huge advantage of having the exact same number of letters), but that would require a database migration on Trac which could be a bit annoying (though not impossible).
If you think that's worth it I can look into that as well.
I've attached a patch for the documentation changes 🙃
(If you insist, I've also opened a PR: https://github.com/django/django/pull/18780)