Skip to content

Remove unnecessary spaces from docs/manual/dynamicform.md#533

Closed
Leemoonsoo wants to merge 1 commit intoapache:masterfrom
Leemoonsoo:ZEPPELIN-490-followup
Closed

Remove unnecessary spaces from docs/manual/dynamicform.md#533
Leemoonsoo wants to merge 1 commit intoapache:masterfrom
Leemoonsoo:ZEPPELIN-490-followup

Conversation

@Leemoonsoo
Copy link
Member

What is this PR for?

#518 introduces some unnecessary spaces in docs/manual/dynamicform.md which results dynamicform.md is not compiled to dynamicform.html under _site directory after bundle exec jekyll build --safe command

What type of PR is it?

Bug Fix

Is there a relevant Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-490

How should this be tested?

run bundle exec jekyll build --safe under /docs and see if /docs/_site/manual/dynamicform.html is generated.

@Leemoonsoo
Copy link
Member Author

Merging if there're no more discussions

@felixcheung
Copy link
Member

LGTM

@AhyoungRyu
Copy link
Contributor

Oh I didn't realized that. It was my bad.
Let's get to merge !

@asfgit asfgit closed this in 9e51f9c Dec 15, 2015
r-kamath pushed a commit to r-kamath/zeppelin that referenced this pull request Dec 16, 2015
### What is this PR for?
apache#518 introduces some unnecessary spaces in docs/manual/dynamicform.md which results dynamicform.md is not compiled to dynamicform.html under _site directory after `bundle exec jekyll build --safe` command

### What type of PR is it?
Bug Fix

### Is there a relevant Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-490

### How should this be tested?
run `bundle exec jekyll build --safe` under /docs and see if /docs/_site/manual/dynamicform.html is generated.

Author: Lee moon soo <moon@apache.org>

Closes apache#533 from Leemoonsoo/ZEPPELIN-490-followup and squashes the following commits:

a06475f [Lee moon soo] Remove unnecessary spaces
asfgit pushed a commit that referenced this pull request Jun 13, 2016
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