Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Sep 16, 2016

What changes were proposed in this pull request?

Fix <ul> / <li> problems in SQL scaladoc.

How was this patch tested?

Scaladoc build and manual verification of generated HTML.

@HyukjinKwon
Copy link
Member

HyukjinKwon commented Sep 16, 2016

Hi @srowen, I support this PR. It's a super minor but would this be sensible if we re-indent the child elements in the HTML in newly added parents?

@SparkQA
Copy link

SparkQA commented Sep 16, 2016

Test build #65482 has finished for PR 15117 at commit 0627254.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 16, 2016

Test build #65484 has finished for PR 15117 at commit 5c9e486.

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

@rxin
Copy link
Contributor

rxin commented Sep 16, 2016

LGTM - merging in master/2.0.

@asfgit asfgit closed this in b9323fc Sep 16, 2016
@rxin
Copy link
Contributor

rxin commented Sep 16, 2016

I merged this in master but there's a conflict with branch-2.0. Can you create a patch for 2.0?

asfgit pushed a commit that referenced this pull request Sep 17, 2016
Fix `<ul> / <li>` problems in SQL scaladoc.

Scaladoc build and manual verification of generated HTML.

Author: Sean Owen <sowen@cloudera.com>

Closes #15117 from srowen/SPARK-17561.

(cherry picked from commit b9323fc)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen srowen deleted the SPARK-17561 branch September 17, 2016 11:43
@srowen
Copy link
Member Author

srowen commented Sep 17, 2016

Sure, I just merged it to 2.0 and resolved the conflict because that was straightforward here.

wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
## What changes were proposed in this pull request?

Fix `<ul> / <li>` problems in SQL scaladoc.

## How was this patch tested?

Scaladoc build and manual verification of generated HTML.

Author: Sean Owen <sowen@cloudera.com>

Closes apache#15117 from srowen/SPARK-17561.
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.

4 participants