Skip to content

[MRG + 1] Changed Contributor's Guide to Development Guide #7690#7691

Merged
jnothman merged 2 commits intoscikit-learn:masterfrom
aferritto:doc-update-7690
Oct 18, 2016
Merged

[MRG + 1] Changed Contributor's Guide to Development Guide #7690#7691
jnothman merged 2 commits intoscikit-learn:masterfrom
aferritto:doc-update-7690

Conversation

@aferritto
Copy link
Copy Markdown
Contributor

Reference Issue

#7690

What does this implement/fix? Explain your changes.

Changed the name of the link to the developer's guide to "development" from "contributor's".

Any other comments?

Could also change name to "Developer's Guide" to match what the linked to page says if that would be preferred. I ran the make and make flake8-diff tests and got some warnings/errors, even though I never changed the code. Can post the exact messages if anyone wants them but figured I'd open this anyways to get this started.

@amueller
Copy link
Copy Markdown
Member

This one is also good, but in the issue I meant here:
http://scikit-learn.org/stable/documentation.html

@amueller
Copy link
Copy Markdown
Member

LGTM.

@amueller amueller changed the title [MRG] Changed Contributor's Guide to Development Guide #7690 [MRG + 1] Changed Contributor's Guide to Development Guide #7690 Oct 18, 2016
@aferritto
Copy link
Copy Markdown
Contributor Author

Looks like CircleCI is timing out

@jnothman
Copy link
Copy Markdown
Member

Thanks @FERRIA

@jnothman jnothman merged commit 647b7c1 into scikit-learn:master Oct 18, 2016
@aferritto aferritto deleted the doc-update-7690 branch October 18, 2016 23:28
amueller pushed a commit to amueller/scikit-learn that referenced this pull request Oct 25, 2016
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
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