8000 add html-noplot and changed help message to make.bat by EFerriss · Pull Request #8524 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

add html-noplot and changed help message to make.bat #8524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 4, 2017
Merged

add html-noplot and changed help message to make.bat #8524

merged 3 commits into from
Mar 4, 2017

Conversation

EFerriss
Copy link
Contributor
@EFerriss EFerriss commented Mar 4, 2017

Reference Issue

What does this implement/fix? Explain your changes.

Windows users who want to make changes to the documentation and check that the html looks correct can now run 'make html-noplot' so that it doesn't go through all of the examples, which would take a long time. The default for make without a target is still to show help (maybe that should change?), but I added a mention of html-noplot to the help message.

Any other comments?

doc/make.bat Outdated
@@ -24,6 +24,7 @@ if "%1" == "help" (
echo. changes to make an overview over all changed/added/deprecated items
echo. linkcheck to check all external links for integrity
echo. doctest to run all doctests embedded in the documentation if enabled
echo. html-noplot to make HTML files using Windows
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the indentation is off here. Can you fix that please?

@EFerriss
Copy link
Contributor Author
EFerriss commented Mar 4, 2017

Done?

@amueller
Copy link
Member
amueller commented Mar 4, 2017

LGTM. merging as minor :)

@amueller amueller merged commit fad5f9b into scikit-learn:master Mar 4, 2017
@tashay
Copy link
Contributor
tashay commented Mar 4, 2017

Congrats @EFerriss, you did it!!!

@Przemo10 Przemo10 mentioned this pull request Mar 17, 2017
herilalaina pushed a commit to herilalaina/scikit-learn that referenced this pull request Mar 26, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
massich pushed a commit to massich/scikit-learn that referenced this pull request Apr 26, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
* add html-noplot and help message to make.bat

* changed spaces to tab in make.bat help

* changed all spaces to tabs in make.bat update
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
0