GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2252
[ZEPPELIN-1940] lint rule set is NOT applied at all. ### What is this PR for? eslint ruleset is NOT applied at all due to invalid `.eslintrc`. This PR includes - fixes which is automatically done by eslint (`eslint --fix src`) - modification in `.eslintrc`, `package.json` - while removing eslint conf in `gruntfile.js` As a result of this PR, we can have more strict and modernized rulesets which can help us to prevent bugs. ### What type of PR is it? [Improvement] ### Todos * [x] - Setup rules * [x] - Remove grunt eslint config * [x] - Ignore useless rules * [x] - Fix code for applied rules ### What is the Jira issue? [ZEPPELIN-1940](https://issues.apache.org/jira/browse/ZEPPELIN-1940) ### How should this be tested? - `cd zeppelin-web` - `npm run lint:once` ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? - NO * Is there breaking changes for older versions? - NO * Does this needs documentation? - NO You can merge this pull request into a Git repository by running: $ git pull https://github.com/1ambda/zeppelin ZEPPELIN-1940/apply-lint-rule-set Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/2252.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2252 ---- commit fcf7569c0b3eb2bbccb33ce531cee33e8ffdd950 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T00:40:25Z fix: Use eslint instead of grunt-eslint commit 1ee0af31976c2d104fedb9c00345ea0201a6eda1 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T00:40:51Z fix: Remove invalid visdev commit ff30ff0864c6dae7eff21f959b084176a6f6ac21 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:08:06Z fix: Set ignored, warn lint rules commit 2d80425af198beab28754e57cc467a9aed0e2818 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:19:51Z fix: ignore all rules commit 065801aad09160e69f12ea21ed3f449898b2beb5 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:24:15Z fix: yoda ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/handsontable/handsonHelper.js 160:49 error Expected literal to be on the right side of === yoda ``` commit 1f1862f74f41a981dbc5ec805dc690b9c993f7b9 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:25:44Z fix: no-mixed-spaces-and-tabs ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-piechart.js (3/0) â 79:2 Mixed spaces and tabs no-mixed-spaces-and-tabs â 80:2 Mixed spaces and tabs no-mixed-spaces-and-tabs â 81:2 Mixed spaces and tabs no-mixed-spaces-and-tabs ``` commit 139b28d96883b4056df8fce3a5ddd979900fae4e Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:27:18Z fix: semi-spacing ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/search/result-list.controller.js (1/0) â 20:26 Unexpected whitespace before semicolon semi-spacing ``` commit deb679cd282db0e40c488a0ae2c31a5dca37f77c Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:31:05Z fix: no-extra-boolean-cast ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/app.js (1/0) â 40:12 Redundant double negation no-extra-boolean-cast /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/jobmanager/jobs/job.controller.js (2/0) â 58:18 Redundant double negation no-extra-boolean-cast â 90:18 Redundant double negation no-extra-boolean-cast ``` commit 8b540e230802490807f1d938baa30f4a3c75c727 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:35:15Z fix: object-property-newline ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js (7/0) â 45:26 Object properties must go on a new line if they aren't all on the same line object-property-newline â 46:24 Object properties must go on a new line if they aren't all on the same line object-property-newline â 46:40 Object properties must go on a new line if they aren't all on the same line object-property-newline â 71:38 Object properties must go on a new line if they aren't all on the same line object-property-newline â 76:36 Object properties must go on a new line if they aren't all on the same line object-property-newline â 89:36 Object properties must go on a new line if they aren't all on the same line object-property-newline â 102:36 Object properties must go on a new line if they aren't all on the same line object-property-newline /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-nvd3chart.js (0/2) â 132:37 Unexpected mix of '&&' and '||' no-mixed-operators â 132:54 Unexpected mix of '&&' and '||' no-mixed-operators /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/helium/helium-conf.js (3/0) â 34:19 Object properties must go on a new line if they aren't all on the same line object-property-newline â 34:41 Object properties must go on a new line if they aren't all on the same line object-property-newline â 35:21 Object properties must go on a new line if they aren't all on the same line object-property-newline ``` commit 685878446dcce8c7134f46c8a489085e4c927fd2 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:38:50Z fix: dot-location ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/configuration/configuration.controller.js (2/0) â 25:67 Expected dot to be on same line as property dot-location â 28:7 Expected dot to be on same line as property dot-location /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/credential/credential.controller.js (8/0) â 28:59 Expected dot to be on same line as property dot-location â 34:7 Expected dot to be on same line as property dot-location â 67:74 Expected dot to be on same line as property dot-location â 78:7 Expected dot to be on same line as property dot-location â 142:69 Expected dot to be on same line as property dot-location â 147:7 Expected dot to be on same line as property dot-location â 169:78 Expected dot to be on same line as property dot-location â 174:13 Expected dot to be on same line as property dot-location /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/helium/helium.controller.js (4/0) â 135:73 Expected dot to be on same line as property dot-location â 139:13 Expected dot to be on same line as property dot-location â 262:38 Expected dot to be on same line as property dot-location â 266:13 Expected dot to be on same line as property dot-location ``` commit fc09077291e178b097ace416eeb2b5bcfe571a28 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:41:30Z fix: brace-style ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js (1/0) â 67:39 Closing curly brace should be on the same line as opening curly brace or on the line after the previous block brace-style /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (1/0) â 228:66 Closing curly brace should be on the same line as opening curly brace or on the line after the previous block brace-style ``` commit ea787650f4c0afddc998be05524886e576ea8432 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:43:24Z fix: one-var ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (1/0) â 210:7 Split initialized 'var' declarations into multiple statements one-var ``` commit 721243ae39e76fda92898ddcb659c5ec30d2d720 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:45:06Z fix: no-multi-spaces ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/notebook/paragraph/result/result.controller.js (4/0) â 403:21 Multiple spaces found before 'elem' no-multi-spaces â 415:21 Multiple spaces found before 'elem' no-multi-spaces â 437:21 Multiple spaces found before 'elem' no-multi-spaces â 715:28 Multiple spaces found before '(' no-multi-spaces /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/tabledata/pivot.js (1/0) â 247:49 Multiple spaces found before 'p' no-multi-spaces /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (1/0) â 271:42 Multiple spaces found before '','' no-multi-spaces ``` commit 4f0391dba7057f3db38040c68788309e7ca53745 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:46:51Z fix: no-empty ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-nvd3chart.js (1/2) â 52:25 Empty block statement no-empty ``` commit b1613fc0ce499b415af7f066c29e573cae2fb94a Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:48:21Z fix: space-in-parens ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (7/0) â 212:10 There should be no spaces inside this paren space-in-parens â 212:18 There should be no spaces inside this paren space-in-parens â 215:10 There should be no spaces inside this paren space-in-parens â 215:18 There should be no spaces inside this paren space-in-parens â 232:47 There should be no spaces inside this paren space-in-parens â 233:10 There should be no spaces inside this paren space-in-parens â 234:74 There should be no spaces inside this paren space-in-parens ``` commit 303863554feb2fc37466e831cb32514a95bb2a11 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:49:31Z fix: keyword-spacing ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/notebook/paragraph/result/result.controller.js (1/0) â 335:11 Expected space(s) after "for" keyword-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/spell/spell-result.js (1/0) â 90:5 Expected space(s) after "for" keyword-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (1/0) â 236:9 Expected space(s) after "else" keyword-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/helium/helium-conf.js (2/0) â 28:3 Expected space(s) after "for" keyword-spacing â 80:5 Expected space(s) after "catch" keyword-spacing ``` commit 6fee1dede56dfbf6542e0db713c0c264fe980b9a Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:52:51Z fix: space-before-blocks ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (6/0) â 81:30 Missing space before opening brace space-before-blocks â 212:17 Missing space before opening brace space-before-blocks â 215:17 Missing space before opening brace space-before-blocks â 228:33 Missing space before opening brace space-before-blocks â 232:47 Missing space before opening brace space-before-blocks â 234:74 Missing space before opening brace space-before-blocks /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/navbar/navbar.controller.js (1/0) â 84:25 Missing space before opening brace space-before-blocks ``` commit a54f85604a067a63bf789e24d3803641a79ebe70 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:55:33Z fix: block-spacing ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/search/result-list.controller.js (2/0) â 148:42 Requires a space after '{' block-spacing â 148:68 Requires a space before '}' block-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-areachart.js (4/0) â 61:40 Requires a space after '{' block-spacing â 61:86 Requires a space before '}' block-spacing â 62:40 Requires a space after '{' block-spacing â 62:72 Requires a space before '}' block-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-barchart.js (2/0) â 61:40 Requires a space after '{' block-spacing â 61:72 Requires a space before '}' block-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-linechart.js (2/0) â 71:40 Requires a space after '{' block-spacing â 71:86 Requires a space before '}' block-spacing ``` commit e7f21d242ea5adc84b3d684892f9171a40922315 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:56:55Z fix: operator-linebreak ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/helium/helium.controller.js (1/0) â 324:78 '?' should be placed at the beginning of the line operator-linebreak /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js (1/0) â 1263:71 '?' should be placed at the beginning of the line operator-linebreak ``` commit 638daee153ae61351b420e9a9cdcbad963267df0 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:58:19Z fix: comma-spacing ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/tabledata/pivot.js (2/0) â 151:29 A space is required after ',' comma-spacing â 156:29 A space is required after ',' comma-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-nvd3chart.js (1/2) â 96:35 A space is required after ',' comma-spacing /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (2/0) â 228:30 A space is required after ',' comma-spacing â 235:32 A space is required after ',' comma-spacing ``` commit 4af2a01d6451a929b63f54ad468241902b5ec5c3 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T15:59:45Z fix: spaced-comment ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/navbar/navbar.controller.js (2/0) â 91:5 Expected space or tab after '//' in comment spaced-comment â 94:7 Expected space or tab after '//' in comment spaced-comment /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/ngenter/ngenter.directive.test.js (2/0) â 19:3 Expected space or tab after '//' in comment spaced-comment â 20:3 Expected space or tab before '*/' in comment spaced-comment /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteName-create/notename.controller.js (1/0) â 102:5 Expected space or tab after '//' in comment spaced-comment /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteName-import/notenameImport.controller.js (1/0) â 122:7 Expected space or tab after '//' in comment spaced-comment /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/saveAs/saveAs.service.js (2/0) â 34:7 Expected space or tab after '//' in comment spaced-comment â 35:7 Expected space or tab after '//' in comment spaced-comment /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/searchService/search.service.js (1/0) â 23:20 Expected space or tab after '//' in comment spaced-comment ``` commit 6f61bc388b7469fb363737d148f3538a524631e5 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:00:53Z fix: no-trailing-spaces ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/helium/helium.controller.js (3/0) â 44:1 Trailing spaces not allowed no-trailing-spaces â 48:1 Trailing spaces not allowed no-trailing-spaces â 111:1 Trailing spaces not allowed no-trailing-spaces ``` commit 631495473b73498f2b0f7ffb6346f4568d1d388b Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:01:52Z fix: no-multiple-empty-lines ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/resizable/resizable.directive.js (1/0) â 70:1 Too many blank lines at the end of file. Max of 0 allowed no-multiple-empty-lines /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/saveAs/saveAs.service.js (1/0) â 52:1 Too many blank lines at the end of file. Max of 0 allowed no-multiple-empty-lines /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/searchService/search.service.js (1/0) â 34:1 Too many blank lines at the end of file. Max of 0 allowed no-multiple-empty-lines /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/websocketEvents/websocketEvents.factory.js (1/0) â 196:1 Too many blank lines at the end of file. Max of 0 allowed no-multiple-empty-lines /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/websocketEvents/websocketMsg.service.js (1/0) â 341:1 Too many blank lines at the end of file. Max of 0 allowed no-multiple-empty-lines ``` commit f9993dc1e030432bd9edc5f1831fc61e257acf73 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:09:08Z fix: indent ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (7/0) â 71:7 Expected indentation of 4 spaces but found 6 indent â 75:7 Expected indentation of 4 spaces but found 6 indent â 82:6 Expected indentation of 4 spaces but found 5 indent â 83:7 Expected indentation of 7 spaces but found 6 indent â 85:7 Expected indentation of 7 spaces but found 6 indent â 87:4 Expected indentation of 2 spaces but found 3 indent â 231:12 Expected indentation of 8 spaces but found 11 indent ``` commit 47636d6742c3bfebbfd0105b5d91fe884f09c26c Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:13:36Z fix: space-before-function-paren ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/websocketEvents/websocketMsg.service.js (48/0) â 17:25 Missing space before function parentheses space-before-function-paren â 22:26 Missing space before function parentheses space-before-function-paren â 26:29 Missing space before function parentheses space-before-function-paren â 36:30 Missing space before function parentheses space-before-function-paren â 40:32 Missing space before function parentheses space-before-function-paren â 44:26 Missing space before function parentheses space-before-function-paren â 48:28 Missing space before function parentheses space-before-function-paren â 52:25 Missing space before function parentheses space-before-function-paren â 56:25 Missing space before function parentheses space-before-function-paren â 60:27 Missing space before function parentheses space-before-function-paren â 64:25 Missing space before function parentheses space-before-function-paren â 68:24 Missing space before function parentheses space-before-function-paren ``` commit 651c5df4e183a04ed3e7867623a53777ad276abb Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:15:41Z fix: ignore object-curly-spacing commit e40edd9ef226a3f603d08a6b918adf9e58e70762 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:16:50Z fix: ignore arrow-parens commit 5920acaeff0eb36845db354ff49301cbdc3ffd9d Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:17:52Z fix: no-extra-semi ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-table.js (5/0) â 28:4 Unnecessary semicolon no-extra-semi â 32:4 Unnecessary semicolon no-extra-semi â 51:4 Unnecessary semicolon no-extra-semi â 57:4 Unnecessary semicolon no-extra-semi â 61:4 Unnecessary semicolon no-extra-semi /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/visualization.js (14/0) â 25:4 Unnecessary semicolon no-extra-semi â 32:4 Unnecessary semicolon no-extra-semi â 39:4 Unnecessary semicolon no-extra-semi â 46:4 Unnecessary semicolon no-extra-semi â 54:4 Unnecessary semicolon no-extra-semi â 64:4 Unnecessary semicolon no-extra-semi â 75:4 Unnecessary semicolon no-extra-semi â 82:4 Unnecessary semicolon no-extra-semi â 89:4 Unnecessary semicolon no-extra-semi â 100:4 Unnecessary semicolon no-extra-semi â 112:4 Unnecessary semicolon no-extra-semi â 119:4 Unnecessary semicolon no-extra-semi â 163:4 Unnecessary semicolon no-extra-semi â 171:2 Unnecessary semicolon no-extra-semi ``` commit faea17a0d66eaa9c4bcc62eb697285482d2529e9 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:19:16Z fix: space-infix-ops ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js (5/0) â 70:54 Infix operators must be spaced space-infix-ops â 74:54 Infix operators must be spaced space-infix-ops â 229:52 Infix operators must be spaced space-infix-ops â 230:53 Infix operators must be spaced space-infix-ops â 231:27 Infix operators must be spaced space-infix-ops /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/helium/helium.service.js (1/0) â 108:33 Infix operators must be spaced space-infix-ops ``` commit 829e398066ad05085f5b90eb52f1131aede3c3d4 Author: 1ambda <1am...@gmail.com> Date: 2017-04-14T16:21:52Z fix: padded-blocks ``` /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteListDataFactory/noteList.datafactory.test.js (2/0) â 1:43 Block must not be padded by blank lines padded-blocks â 77:1 Block must not be padded by blank lines padded-blocks /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteName-create/notename.controller.js (1/0) â 105:1 Block must not be padded by blank lines padded-blocks /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteName-create/notename.controller.test.js (1/0) â 40:1 Block must not be padded by blank lines padded-blocks /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/noteName-import/notenameImport.controller.js (1/0) â 114:5 Block must not be padded by blank lines padded-blocks /Users/1ambda/github/apache/apache-zeppelin/zeppelin-master/zeppelin-web/src/components/websocketEvents/websocketEvents.factory.js (1/0) â 110:5 Block must not be padded by blank lines padded-blocks ``` ---- --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---