Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#18971 closed Cleanup/optimization (fixed)

Add a CONTRIBUTING doc

Reported by: Karen Tracey Owned by: nobody
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

https://github.com/blog/1184-contributing-guidelines

Seems this would be helpful to point people in the right direction as to when it's best to open a ticket vs. just opening a pull request.

Change History (7)

comment:1 by Alex Gaynor, 12 years ago

Since we already have an actual document (in a real documentation system), let's just have a CONTRIBUTING file that has a link to the real docs.

comment:2 by Alex Gaynor, 12 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Preston Holmes, 12 years ago

I concur with Alex - I feel the current contributing docs are actually quite good, and we just need to point people there.

I've put together a draft, and verified that there is no problem using an .rst document

https://gist.github.com/3741235

comment:4 by Preston Holmes <preston@…>, 12 years ago

Resolution: fixed
Status: newclosed

In 70248cce0a8a3144df9f36e15fbeb5cc1b091c3e:

Fixed #18971 -- added root level CONTRIBUTING doc

comment:5 by Karen Tracey, 12 years ago

Resolution: fixed
Status: closedreopened

This isn't showing up on github as described in the referenced github blog entry. That entry only mentions the names CONTRIBUTING and CONTRIBUTING.md, ours is CONTRIBUTING.rst, possibly that is the problem?

comment:6 by Preston Holmes, 12 years ago

Resolution: fixed
Status: reopenedclosed

I had tested this with one of my other repos before pushing and .rst isn't the issue - after another quick experiment - it looks like the issue is that you need the contributing doc in the branch from where you are opening the pull request. I'd say that is a bit of a shortcoming of the github feature, but not in our control.

If you git cherry-pick 70248cce0a8a3144df9f36e15fbeb5cc1b091c3e from your branch (assuming your master is current), push the branch, then attempt to open the pull request, you'll see the banner, the link is to the version of the doc in your fork, another reason it is nice to refer to our published docs - because they can be current even if the fork was stale.

comment:7 by Karen Tracey, 12 years ago

Ah. Thanks for the explanation...

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