Skip to content

[MINOR] [DOCS] fixed list display in ml-ensembles #10025

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[MINOR] [DOCS] fixed list display in ml-ensembles #10025

wants to merge 1 commit into from

Conversation

@BenFradet
Copy link
Contributor

@BenFradet BenFradet commented Nov 28, 2015

The list in ml-ensembles.md wasn't properly formatted and, as a result, was looking like this:
old

This PR aims to make it look like this:
new

@SparkQA
Copy link

SparkQA commented Nov 28, 2015

Test build #46842 has finished for PR 10025 at commit a9829a9.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@BenFradet
Copy link
Contributor Author

BenFradet commented Nov 30, 2015

Pinging @mengxr

@srowen
Copy link
Member

srowen commented Nov 30, 2015

LGTM

@BenFradet
Copy link
Contributor Author

BenFradet commented Nov 30, 2015

Thanks for having a look @srowen.

@mengxr
Copy link
Contributor

mengxr commented Nov 30, 2015

Merged into master and branch-1.6. Thanks!

@asfgit asfgit closed this in f2fbfa4 Nov 30, 2015
asfgit pushed a commit that referenced this pull request Nov 30, 2015
@BenFradet @mengxr
[MINOR][DOCS] fixed list display in ml-ensembles
0978ec1 
The list in ml-ensembles.md wasn't properly formatted and, as a result, was looking like this:
![old](http://i.imgur.com/2ZhELLR.png)

This PR aims to make it look like this:
![new](http://i.imgur.com/0Xriwd2.png)

Author: BenFradet <[email protected]>

Closes #10025 from BenFradet/ml-ensembles-doc.

(cherry picked from commit f2fbfa4)
Signed-off-by: Xiangrui Meng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@BenFradet @SparkQA @srowen @mengxr