Skip to content

[ZEPPELIN-799] Modify index.html and karma.conf.js for mode-python#851

Closed
cyhsutw wants to merge 1 commit intoapache:masterfrom
cyhsutw:update-template-files
Closed

[ZEPPELIN-799] Modify index.html and karma.conf.js for mode-python#851
cyhsutw wants to merge 1 commit intoapache:masterfrom
cyhsutw:update-template-files

Conversation

@cyhsutw
Copy link
Copy Markdown
Contributor

@cyhsutw cyhsutw commented Apr 22, 2016

What is this PR for?

Python edit mode was added in #825 thank to @astroshim.

I think we'll need to update the src/index.html and test/karma.conf.js under zeppelin-web accordingly to avoid unstaged changes after building.

What type of PR is it?

Improvement

Todos

None

What is the Jira issue?

Probably a subtask of
https://issues.apache.org/jira/browse/ZEPPELIN-799

How should this be tested?

Shouldn't be any since #825 is merged into master and has been tested.

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

/cc @astroshim

@cyhsutw cyhsutw closed this Apr 22, 2016
@cyhsutw cyhsutw reopened this Apr 22, 2016
@prabhjyotsingh
Copy link
Copy Markdown
Contributor

LGTM

@felixcheung
Copy link
Copy Markdown
Member

right, thanks.

@prabhjyotsingh
Copy link
Copy Markdown
Contributor

Merging if there's no more discussion

@asfgit asfgit closed this in 9af256e Apr 24, 2016
@cyhsutw cyhsutw deleted the update-template-files branch April 24, 2016 21:58
onkarshedge pushed a commit to onkarshedge/incubator-zeppelin that referenced this pull request May 11, 2016
### What is this PR for?

Python edit mode was added in apache#825 thank to astroshim.

I think we'll need to update the `src/index.html` and `test/karma.conf.js` under `zeppelin-web` accordingly to avoid unstaged changes after building.

### What type of PR is it?
Improvement

### Todos
None

### What is the Jira issue?
Probably a subtask of
https://issues.apache.org/jira/browse/ZEPPELIN-799

### How should this be tested?
Shouldn't be any since apache#825 is merged into `master` and has been tested.

### Screenshots (if appropriate)
N/A

### Questions:
* Does the licenses files need update? *no*
* Is there breaking changes for older versions? *no*
* Does this needs documentation? *no*

/cc astroshim

Author: Cheng-Yu Hsu <[email protected]>

Closes apache#851 from cyhsutw/update-template-files and squashes the following commits:

24901cc [Cheng-Yu Hsu] add `mode-python` to generated html and js
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