Skip to content

Update contributing guide#3832

Merged
CyberShadow merged 1 commit intodlang:masterfrom
JackStouffer:patch-5
Dec 3, 2015
Merged

Update contributing guide#3832
CyberShadow merged 1 commit intodlang:masterfrom
JackStouffer:patch-5

Conversation

@JackStouffer
Copy link
Contributor

  • Added a FAQ section to answer some common questions from people have about their PRs. This was inspired about by Tuple's toString made useful #3594, where a lot of good work was almost lost.
  • fixed the links to be in a list rather than headers

@Hackerpilot
Copy link
Contributor

@CyberShadow
Copy link
Member

I think this should be on the wiki, since it applies to all projects.

@JackStouffer
Copy link
Contributor Author

Ok, but I think it should be in here as well, because I think more people read CONTRIBUTING.md as GitHub auto links to it.

@CyberShadow
Copy link
Member

Then it should link to the wiki. DRY.

@CyberShadow
Copy link
Member

To clarify...

We seem to have a plethora of contribution guides. Everyone seems to want to write a contribution guide, because it's going to be better than all the other ones or something. As a result we have an uncoordinated duplication of information and effort. This can lead to situations such as "OK, how did you manage to create this git/PR/etc. abomination?" "I followed the guide." "Which guide?" "The one on the D website." ...

First, there's the wiki pages which are already linked to from this file:

http://wiki.dlang.org/Starting_as_a_Contributor
http://wiki.dlang.org/Pull_Requests
http://wiki.dlang.org/Building_DMD

More D Wiki pages:

http://wiki.dlang.org/Get_involved
http://wiki.dlang.org/Phobos_Quality
http://wiki.dlang.org/How_You_Can_Help

...and other pages on http://wiki.dlang.org/Category:Contribution_Guidelines.

Then, there's the projects' .md files:

https://github.com/D-Programming-Language/druntime/blob/master/HACKING.md
https://github.com/D-Programming-Language/dlang.org/blob/master/CONTRIBUTING.md

If anything, we need to turn the valve in the other direction, and work on consolidation and quality, not quantity.

@JackStouffer
Copy link
Contributor Author

Well, I just created this page http://wiki.dlang.org/Contributing_FAQ and linked it in the file

@CyberShadow
Copy link
Member

No, as I wrote on the talk page please do not create any more pages. Add the information to one of the many existing pages instead.

@JackStouffer
Copy link
Contributor Author

I'll add it to the "Starting as a Contributor" page, butI don't have permission to delete pages.

@CyberShadow
Copy link
Member

Can you integrate any useful information into http://wiki.dlang.org/Pull_Requests ? It already covers the topic of rebasing.

@JackStouffer
Copy link
Contributor Author

@CyberShadow I'd rather not write a tutorial on squashing or updating with rebase as I think the tutorials I link do a better job than I could of explaining things.

@CyberShadow
Copy link
Member

Are those tutorials significantly better than the information we have on the wiki? We should avoid relying on 3rd-party resources anyway, as they're not updatable and subject to bitrot.

@JackStouffer
Copy link
Contributor Author

Updated link

Are those tutorials significantly better than the information we have on the wiki

Yes, there is nothing that I see in "Pull Requests" that explains squashing and I think the tutorial for rebasing does a much better job of explaining rebasing for updating PRs.

@CyberShadow
Copy link
Member

OK, then how about adding the links to http://wiki.dlang.org/Pull_Requests?

@JackStouffer
Copy link
Contributor Author

@CyberShadow
Copy link
Member

That would be great if you want to undertake such a project. (But of course, only if the result is not less informative than its parts.)

@JackStouffer
Copy link
Contributor Author

Sure. Can you delete http://wiki.dlang.org/Contributing_FAQ

@CyberShadow
Copy link
Member

Done.

@JackStouffer
Copy link
Contributor Author

@CyberShadow I think I've successfully done it http://wiki.dlang.org/Starting_as_a_Contributor

@CyberShadow
Copy link
Member

Haven't reviewed it but good work 👍 You can change the other pages to be redirects to relevant sections to this one, then fix links to the olde pages.

@JackStouffer JackStouffer changed the title Add FAQ to the contributing guide Update contributing guide Dec 1, 2015
@JackStouffer
Copy link
Contributor Author

Updated changes to CONTRIBUTING.md

@CyberShadow
Copy link
Member

@CyberShadow
Copy link
Member

And another page that needs merging: http://wiki.dlang.org/Using_Git_on_Windows

@JackStouffer
Copy link
Contributor Author

I will fix the redirects and merge the other article, but I believe this can be merged regardless.

@CyberShadow
Copy link
Member

Can we get rid of the single-item-list? Maybe integrate the link into the above paragraph, e.g. "To get started, please read the Starting as a Contributor article on the D wiki."

@JackStouffer
Copy link
Contributor Author

Fixed

@CyberShadow
Copy link
Member

(But of course, only if the result is not less informative than its parts.)

Seems the new page is less informative than its parts. For example, there is no mention of Digger or the other build tools.

@CyberShadow
Copy link
Member

Auto-merge toggled on

@JackStouffer
Copy link
Contributor Author

Added the existing tools section onto the page.

@JackStouffer
Copy link
Contributor Author

Auto-merge toggled on

Thanks

@CyberShadow
Copy link
Member

Added the existing tools section onto the page.

Can you please make sure that nothing else was missed?

CyberShadow added a commit that referenced this pull request Dec 3, 2015
@CyberShadow CyberShadow merged commit d188c09 into dlang:master Dec 3, 2015
@JackStouffer JackStouffer deleted the patch-5 branch March 29, 2016 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants