[12/12] zeppelin git commit: [ZEPPELIN-3245] checkstyle/eslintrc for zeppelin-web (JavaScript)

2018-02-22 Thread prabhjyotsingh 
[ZEPPELIN-3245] checkstyle/eslintrc for zeppelin-web (JavaScript)

Have added this PR to add a rule in eslinerc to have semicolons in javascript 
source

[Improvement | Refactoring]

* [ZEPPELIN-3245](https://issues.apache.org/jira/browse/ZEPPELIN-3245)

```
cd zeppelin-web
npm install (or yarn install if you have yarn)
npm run lint:once
```

Author: Prabhjyot Singh 

Closes #2804 from prabhjyotsingh/discuss/eslint_semi_rule and squashes the 
following commits:

4506f243e [Prabhjyot Singh] eslint rule for space
bc43d68a6 [Prabhjyot Singh] merge `[ZEPPELIN-3177]Resize charts on paragaph 
resize` changes
2d57ba30b [Prabhjyot Singh] fix failing WEB_E2E="true"
f23cb61d4 [Prabhjyot Singh] remove `"linebreak-style": 0,` and 
`"no-use-before-define": 0,`
39f37fb88 [Prabhjyot Singh] remove "standard" from eslint
6edac44f6 [Prabhjyot Singh] add `"semi": [2, "always"]` rule in eslinerc

Change-Id: I91546ea973c2c9e7540da1586d6329fc93088eb0
(cherry picked from commit ea2c944742cea6e8e37e225d1acc67cdb195056e)
Signed-off-by: Prabhjyot Singh 


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/ed517a20
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/ed517a20
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/ed517a20

Branch: refs/heads/branch-0.8
Commit: ed517a2001cabe60ed79cd4c45fc94da4854c436
Parents: 5ee1f79
Author: Prabhjyot Singh 
Authored: Fri Feb 23 09:35:30 2018 +0530
Committer: Prabhjyot Singh 
Committed: Fri Feb 23 10:24:54 2018 +0530

--
 .../org/apache/zeppelin/AbstractZeppelinIT.java |2 +-
 zeppelin-web/.eslintrc  |   24 +-
 zeppelin-web/src/app/app.controller.js  |   52 +-
 zeppelin-web/src/app/app.controller.test.js |   44 +-
 zeppelin-web/src/app/app.js |  162 +-
 .../configuration/configuration.controller.js   |   42 +-
 .../src/app/configuration/configuration.test.js |   92 +-
 .../src/app/credential/credential.controller.js |  236 +-
 .../src/app/credential/credential.test.js   |  132 +-
 zeppelin-web/src/app/helium/helium-conf.js  |   96 +-
 zeppelin-web/src/app/helium/helium-package.js   |   32 +-
 zeppelin-web/src/app/helium/helium-type.js  |2 +-
 .../src/app/helium/helium.controller.js |  424 ++--
 zeppelin-web/src/app/helium/helium.service.js   |  334 +--
 zeppelin-web/src/app/helium/index.js|4 +-
 zeppelin-web/src/app/home/home.controller.js|  188 +-
 .../interpreter/interpreter-item.directive.js   |   20 +-
 .../app/interpreter/interpreter.controller.js   |  838 +++
 .../src/app/interpreter/interpreter.filter.js   |   12 +-
 .../widget/number-widget.directive.js   |   20 +-
 zeppelin-web/src/app/jobmanager/job-status.js   |   26 +-
 .../src/app/jobmanager/job/job.component.js |  118 +-
 .../app/jobmanager/job/job.component.test.js|   70 +-
 .../src/app/jobmanager/jobmanager.component.js  |  177 +-
 .../app/jobmanager/jobmanager.component.test.js |   34 +-
 .../src/app/jobmanager/jobmanager.filter.js |   38 +-
 .../src/app/jobmanager/jobmanager.service.js|   46 +-
 .../app/jobmanager/jobmanager.service.test.js   |   66 +-
 .../notebook-repository.controller.js   |   78 +-
 .../dropdown-input/dropdown-input.directive.js  |   14 +-
 .../dynamic-forms/dynamic-forms.directive.js|   42 +-
 .../elastic-input/elastic-input.controller.js   |   10 +-
 .../src/app/notebook/note-var-share.service.js  |   36 +-
 .../src/app/notebook/notebook.controller.js | 1590 ++--
 .../app/notebook/notebook.controller.test.js|  240 +-
 .../notebook/paragraph/clipboard.controller.js  |   36 +-
 .../code-editor/code-editor.directive.js|   24 +-
 .../notebook/paragraph/paragraph.controller.js  | 1824 +++---
 .../paragraph/paragraph.controller.test.js  |   68 +-
 .../app/notebook/paragraph/paragraph.status.js  |   16 +-
 .../notebook/paragraph/resizable.directive.js   |   66 +-
 .../paragraph/result/result.controller.js   | 1118 -
 .../revisions-comparator.component.js   |  162 +-
 .../notebook/save-as/browser-detect.service.js  |   26 +-
 .../src/app/notebook/save-as/save-as.service.js |   60 +-
 .../src/app/search/result-list.controller.js|  134 +-
 zeppelin-web/src/app/search/search.service.js   |   24 +-
 zeppelin-web/src/app/spell/index.js |4 +-
 zeppelin-web/src/app/spell/spell-base.js|   14 +-
 zeppelin-web/src/app/spell/spell-result.js  |  182 +-
 .../tabledata/advanced-transformation-util.js   | 1010 
 .../advanced-transformation-util.test.js| 2288 +-
 .../app/tabledata/advanced-transformation.js|  202 +-
 .../src/app/tabledata/columnselector.js |   54 +-

[12/12] zeppelin git commit: [ZEPPELIN-3245] checkstyle/eslintrc for zeppelin-web (JavaScript)

2018-02-22 Thread prabhjyotsingh 
[ZEPPELIN-3245] checkstyle/eslintrc for zeppelin-web (JavaScript)

Have added this PR to add a rule in eslinerc to have semicolons in javascript 
source

[Improvement | Refactoring]

* [ZEPPELIN-3245](https://issues.apache.org/jira/browse/ZEPPELIN-3245)

```
cd zeppelin-web
npm install (or yarn install if you have yarn)
npm run lint:once
```

Author: Prabhjyot Singh 

Closes #2804 from prabhjyotsingh/discuss/eslint_semi_rule and squashes the 
following commits:

4506f243e [Prabhjyot Singh] eslint rule for space
bc43d68a6 [Prabhjyot Singh] merge `[ZEPPELIN-3177]Resize charts on paragaph 
resize` changes
2d57ba30b [Prabhjyot Singh] fix failing WEB_E2E="true"
f23cb61d4 [Prabhjyot Singh] remove `"linebreak-style": 0,` and 
`"no-use-before-define": 0,`
39f37fb88 [Prabhjyot Singh] remove "standard" from eslint
6edac44f6 [Prabhjyot Singh] add `"semi": [2, "always"]` rule in eslinerc

Change-Id: I91546ea973c2c9e7540da1586d6329fc93088eb0


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/ea2c9447
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/ea2c9447
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/ea2c9447

Branch: refs/heads/master
Commit: ea2c944742cea6e8e37e225d1acc67cdb195056e
Parents: f6ef64f
Author: Prabhjyot Singh 
Authored: Fri Feb 23 09:35:30 2018 +0530
Committer: Prabhjyot Singh 
Committed: Fri Feb 23 10:24:33 2018 +0530

--
 .../org/apache/zeppelin/AbstractZeppelinIT.java |2 +-
 zeppelin-web/.eslintrc  |   24 +-
 zeppelin-web/src/app/app.controller.js  |   52 +-
 zeppelin-web/src/app/app.controller.test.js |   44 +-
 zeppelin-web/src/app/app.js |  162 +-
 .../configuration/configuration.controller.js   |   42 +-
 .../src/app/configuration/configuration.test.js |   92 +-
 .../src/app/credential/credential.controller.js |  236 +-
 .../src/app/credential/credential.test.js   |  132 +-
 zeppelin-web/src/app/helium/helium-conf.js  |   96 +-
 zeppelin-web/src/app/helium/helium-package.js   |   32 +-
 zeppelin-web/src/app/helium/helium-type.js  |2 +-
 .../src/app/helium/helium.controller.js |  424 ++--
 zeppelin-web/src/app/helium/helium.service.js   |  334 +--
 zeppelin-web/src/app/helium/index.js|4 +-
 zeppelin-web/src/app/home/home.controller.js|  188 +-
 .../interpreter/interpreter-item.directive.js   |   20 +-
 .../app/interpreter/interpreter.controller.js   |  838 +++
 .../src/app/interpreter/interpreter.filter.js   |   12 +-
 .../widget/number-widget.directive.js   |   20 +-
 zeppelin-web/src/app/jobmanager/job-status.js   |   26 +-
 .../src/app/jobmanager/job/job.component.js |  118 +-
 .../app/jobmanager/job/job.component.test.js|   70 +-
 .../src/app/jobmanager/jobmanager.component.js  |  177 +-
 .../app/jobmanager/jobmanager.component.test.js |   34 +-
 .../src/app/jobmanager/jobmanager.filter.js |   38 +-
 .../src/app/jobmanager/jobmanager.service.js|   46 +-
 .../app/jobmanager/jobmanager.service.test.js   |   66 +-
 .../notebook-repository.controller.js   |   78 +-
 .../dropdown-input/dropdown-input.directive.js  |   14 +-
 .../dynamic-forms/dynamic-forms.directive.js|   42 +-
 .../elastic-input/elastic-input.controller.js   |   10 +-
 .../src/app/notebook/note-var-share.service.js  |   36 +-
 .../src/app/notebook/notebook.controller.js | 1590 ++--
 .../app/notebook/notebook.controller.test.js|  240 +-
 .../notebook/paragraph/clipboard.controller.js  |   36 +-
 .../code-editor/code-editor.directive.js|   24 +-
 .../notebook/paragraph/paragraph.controller.js  | 1824 +++---
 .../paragraph/paragraph.controller.test.js  |   68 +-
 .../app/notebook/paragraph/paragraph.status.js  |   16 +-
 .../notebook/paragraph/resizable.directive.js   |   66 +-
 .../paragraph/result/result.controller.js   | 1118 -
 .../revisions-comparator.component.js   |  162 +-
 .../notebook/save-as/browser-detect.service.js  |   26 +-
 .../src/app/notebook/save-as/save-as.service.js |   60 +-
 .../src/app/search/result-list.controller.js|  134 +-
 zeppelin-web/src/app/search/search.service.js   |   24 +-
 zeppelin-web/src/app/spell/index.js |4 +-
 zeppelin-web/src/app/spell/spell-base.js|   14 +-
 zeppelin-web/src/app/spell/spell-result.js  |  182 +-
 .../tabledata/advanced-transformation-util.js   | 1010 
 .../advanced-transformation-util.test.js| 2288 +-
 .../app/tabledata/advanced-transformation.js|  202 +-
 .../src/app/tabledata/columnselector.js |   54 +-
 zeppelin-web/src/app/tabledata/dataset.js   |6 +-
 .../src/app/tabledata/datasetfactory.js |   12 +-