Opened 5 years ago

Closed 5 years ago

#16777 closed Cleanup/optimization (invalid)

Contributing guide: github and bitbucket pull requests are acceptable

Reported by: Aymeric Augustin Owned by: Justin Lilly
Component: Documentation Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

See Russell's comment here: https://code.djangoproject.com/ticket/16774#comment:2

I'm pretty sure this isn't written in the current documentation.

Change History (14)

comment:1 Changed 5 years ago by Aymeric Augustin

And explain how to download the diff, that's what triggers and core developers need most.

comment:2 Changed 5 years ago by Julien Phalip

Triage Stage: UnreviewedAccepted

comment:3 Changed 5 years ago by Jannis Leidel

Note, a Trac ticket is still needed for pull requests to get attention from the core team. They are only other forms of patches, rather than tickets.

comment:4 Changed 5 years ago by Julien Phalip

See also #16779, which is essentially different but potentially has some overlap with this ticket.

Last edited 5 years ago by Julien Phalip (previous) (diff)

comment:5 Changed 5 years ago by Justin Lilly

Owner: changed from nobody to Justin Lilly
Status: newassigned

comment:6 Changed 5 years ago by Justin Lilly

Has patch: set

First pass at the documentation.

https://github.com/django/django/pull/42

comment:7 Changed 5 years ago by Preston Holmes

Would clarify "will likely" to "discussion should be continued" to make clear that trac is where all discussion should happen. Also need to update https://code.djangoproject.com/wiki/CollaborateOnGithub

comment:8 Changed 5 years ago by Bas Peschier

Patch needs improvement: set

And while you are at it, give GitHub and Bitbucket a few capital letters to reflect their brand names :-)

comment:9 Changed 5 years ago by Justin Lilly

Patch needs improvement: unset

Updated. Same link as before, wiki was also updated.

comment:10 Changed 5 years ago by Bas Peschier

Triage Stage: AcceptedReady for checkin

comment:11 Changed 5 years ago by Jannis Leidel

Patch needs improvement: set
Triage Stage: Ready for checkinAccepted

This needs a bit fat warning in the first paragraph that a Trac ticket is still needed.

comment:12 Changed 5 years ago by Justin Lilly

Updated.

comment:13 Changed 5 years ago by Marc Tamlyn

Link to pull request now refers to a completely different issue after the github move...

comment:14 Changed 5 years ago by Aymeric Augustin

Resolution: invalid
Status: assignedclosed

Indeed, this ticket no longer makes sense now that we've moved to GitHub. We are going to write some GitHub guidelines (eventually). I'm not sure BitBucket pull requests will be accepted.

Note: See TracTickets for help on using tickets.
Back to Top