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 ```
2026-05-24 09:38:26 +00:00 · 2017-04-15 00:59:45 +09:00 · 2017-04-15 00:59:45 +09:00 · ca94341826
commit ca94341826
parent d65c47bb42
18 changed files with 29 additions and 30 deletions
--- a/zeppelin-web/.eslintrc
+++ b/zeppelin-web/.eslintrc
@ -41,7 +41,6 @@
    "no-eval": 0,
    "no-extra-semi": 0,
    "arrow-parens": 0,
-    "spaced-comment": 0,
    "no-unneeded-ternary": 0,
    "no-multiple-empty-lines": 0,
    "no-trailing-spaces": 0,
--- a/zeppelin-web/src/app/app.js
+++ b/zeppelin-web/src/app/app.js
@ -118,7 +118,7 @@ var zeppelinWebApp = angular.module('zeppelinWebApp', [
    });
  })

-  //handel logout on API failure
+  // handel logout on API failure
  .config(function ($httpProvider, $provide) {
    $provide.factory('httpInterceptor', function ($q, $rootScope) {
      return {
--- a/zeppelin-web/src/app/interpreter/interpreter.controller.js
+++ b/zeppelin-web/src/app/interpreter/interpreter.controller.js
@ -440,7 +440,7 @@ function InterpreterCtrl($rootScope, $scope, $http, baseUrlSrv, ngToast, $timeou
  };

  $scope.addNewInterpreterSetting = function() {
-    //user input validation on interpreter creation
+    // user input validation on interpreter creation
    if (!$scope.newInterpreterSetting.name ||
      !$scope.newInterpreterSetting.name.trim() || !$scope.newInterpreterSetting.group) {
      BootstrapDialog.alert({
--- a/zeppelin-web/src/app/notebook/notebook.controller.js
+++ b/zeppelin-web/src/app/notebook/notebook.controller.js
@ -172,13 +172,13 @@ function NotebookCtrl($scope, $route, $routeParams, $location, $rootScope,
    return note ? note.name.split('/')[0] === TRASH_FOLDER_ID : false;
  };

-  //Export notebook
+  // Export notebook
  $scope.exportNote = function() {
    var jsonContent = JSON.stringify($scope.note);
    saveAsService.saveAs(jsonContent, $scope.note.name, 'json');
  };

-  //Clone note
+  // Clone note
  $scope.cloneNote = function(noteId) {
    BootstrapDialog.confirm({
      closable: true,
@ -366,7 +366,7 @@ function NotebookCtrl($scope, $route, $routeParams, $location, $rootScope,
  $scope.startSaveTimer = function() {
    $scope.killSaveTimer();
    $scope.isNoteDirty = true;
-    //console.log('startSaveTimer called ' + $scope.note.id);
+    // console.log('startSaveTimer called ' + $scope.note.id);
    $scope.saveTimer = $timeout(function() {
      $scope.saveNote();
    }, 10000);
@ -961,7 +961,7 @@ function NotebookCtrl($scope, $route, $routeParams, $location, $rootScope,
    var newIndex = -1;
    for (var i = 0; i < $scope.note.paragraphs.length; i++) {
      if ($scope.note.paragraphs[i].id === paragraphId) {
-        //determine position of where to add new paragraph; default is below
+        // determine position of where to add new paragraph; default is below
        if (position === 'above') {
          newIndex = i;
        } else {
@ -996,7 +996,7 @@ function NotebookCtrl($scope, $route, $routeParams, $location, $rootScope,

    initializeLookAndFeel();

-    //open interpreter binding setting when there're none selected
+    // open interpreter binding setting when there're none selected
    getInterpreterBindings();
    getPermissions();
    var isPersonalized = $scope.note.config.personalizedMode;
--- a/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js
+++ b/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js
@ -437,7 +437,7 @@ function ParagraphCtrl($scope, $rootScope, $route, $window, $routeParams, $locat
    var newIndex = -1;
    for (var i = 0; i < $scope.note.paragraphs.length; i++) {
      if ($scope.note.paragraphs[i].id === $scope.paragraph.id) {
-        //determine position of where to add new paragraph; default is below
+        // determine position of where to add new paragraph; default is below
        if (position === 'above') {
          newIndex = i;
        } else {
--- a/zeppelin-web/src/app/notebook/paragraph/result/result.controller.js
+++ b/zeppelin-web/src/app/notebook/paragraph/result/result.controller.js
@ -397,7 +397,7 @@ function ResultCtrl($scope, $rootScope, $route, $window, $routeParams, $location
        elem.find('pre code').each(function(i, e) {
          hljs.highlightBlock(e);
        });
-        /*eslint new-cap: [2, {"capIsNewExceptions": ["MathJax.Hub.Queue"]}]*/
+        /* eslint new-cap: [2, {"capIsNewExceptions": ["MathJax.Hub.Queue"]}] */
        MathJax.Hub.Queue(['Typeset', MathJax.Hub, elem[0]]);
      },
      (error) => { elem.html(`${error.stack}`); }
--- a/zeppelin-web/src/app/spell/spell-base.js
+++ b/zeppelin-web/src/app/spell/spell-base.js
@ -15,12 +15,12 @@
 * limitations under the License.
 */

-/*eslint-disable no-unused-vars */
+/* eslint-disable no-unused-vars */
 import {
  DefaultDisplayType,
  SpellResult,
 } from './spell-result';
-/*eslint-enable no-unused-vars */
+/* eslint-enable no-unused-vars */

 export class SpellBase {
  constructor(magic) {
--- a/zeppelin-web/src/app/tabledata/pivot.js
+++ b/zeppelin-web/src/app/tabledata/pivot.js
@ -250,7 +250,7 @@ export default class PivotTransformation extends Transformation {
      }
    }

-    //console.log('schema=%o, rows=%o', schema, rows);
+    // console.log('schema=%o, rows=%o', schema, rows);
    return {
      keys: keys,
      groups: groups,
--- a/zeppelin-web/src/app/visualization/builtins/visualization-nvd3chart.js
+++ b/zeppelin-web/src/app/visualization/builtins/visualization-nvd3chart.js
@ -121,7 +121,7 @@ export default class Nvd3ChartVisualization extends Visualization {
    };

    var traverse = function(sKey, s, rKey, r, func, rowName, rowValue, colName) {
-      //console.log("TRAVERSE sKey=%o, s=%o, rKey=%o, r=%o, rowName=%o, rowValue=%o, colName=%o", sKey, s, rKey, r, rowName, rowValue, colName);
+      // console.log("TRAVERSE sKey=%o, s=%o, rKey=%o, r=%o, rowName=%o, rowValue=%o, colName=%o", sKey, s, rKey, r, rowName, rowValue, colName);

      if (s.type === 'key') {
        rowName = concat(rowName, sKey);
@ -163,7 +163,7 @@ export default class Nvd3ChartVisualization extends Visualization {

    for (var k in rows) {
      traverse(sKey, schema[sKey], k, rows[k], function(rowName, rowValue, colName, value) {
-        //console.log("RowName=%o, row=%o, col=%o, value=%o", rowName, rowValue, colName, value);
+        // console.log("RowName=%o, row=%o, col=%o, value=%o", rowName, rowValue, colName, value);
        if (rowNameIndex[rowValue] === undefined) {
          rowIndexValue[rowIdx] = rowValue;
          rowNameIndex[rowValue] = rowIdx++;
--- a/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js
+++ b/zeppelin-web/src/app/visualization/builtins/visualization-scatterchart.js
@ -75,7 +75,7 @@ export default class ScatterchartVisualization extends Nvd3ChartVisualization {
      });

    chart.showDistX(true).showDistY(true);
-    //handle the problem of tooltip not showing when muliple points have same value.
+    // handle the problem of tooltip not showing when muliple points have same value.
  };

  yAxisTickFormat(d, yLabels) {
@ -335,7 +335,7 @@ export default class ScatterchartVisualization extends Nvd3ChartVisualization {
      var row = rows[i];
      var size = row[options.size.index];

-      //check if the field is numeric
+      // check if the field is numeric
      if (isNaN(parseFloat(size)) || !isFinite(size)) {
        return false;
      }
@ -350,7 +350,7 @@ export default class ScatterchartVisualization extends Nvd3ChartVisualization {
      }
    }

-    //check if all existing fields are discrete
+    // check if all existing fields are discrete
    var isAllDiscrete = ((options.xAxis && options.yAxis && this.isDiscrete(xValues) && this.isDiscrete(yValues)) ||
    (!options.xAxis && this.isDiscrete(yValues)) ||
    (!options.yAxis && this.isDiscrete(xValues)));
--- a/zeppelin-web/src/components/baseUrl/baseUrl.service.js
+++ b/zeppelin-web/src/components/baseUrl/baseUrl.service.js
@ -23,7 +23,7 @@ function baseUrlSrv() {
        port = 443;
      }
    }
-    //Exception for when running locally via grunt
+    // Exception for when running locally via grunt
    if (port === process.env.WEB_PORT) {
      port = process.env.SERVER_PORT;
    }
--- a/zeppelin-web/src/components/login/login.controller.js
+++ b/zeppelin-web/src/components/login/login.controller.js
@ -39,7 +39,7 @@ function LoginCtrl($scope, $rootScope, $http, $httpParamSerializer, baseUrlSrv,
      $rootScope.userName = $scope.loginParams.userName;
      $scope.SigningIn = false;

-      //redirect to the page from where the user originally was
+      // redirect to the page from where the user originally was
      if ($location.search() && $location.search()['ref']) {
        $timeout(function() {
          var redirectLocation = $location.search()['ref'];
@ -62,7 +62,7 @@ function LoginCtrl($scope, $rootScope, $http, $httpParamSerializer, baseUrlSrv,
    };
  };

-  //handle session logout message received from WebSocket
+  // handle session logout message received from WebSocket
  $rootScope.$on('session_logout', function(event, data) {
    if ($rootScope.userName !== '') {
      $rootScope.userName = '';
--- a/zeppelin-web/src/components/navbar/navbar.controller.js
+++ b/zeppelin-web/src/components/navbar/navbar.controller.js
@ -88,10 +88,10 @@ function NavCtrl($scope, $rootScope, $http, $routeParams, $location,
  function logout() {
    var logoutURL = baseUrlSrv.getRestApiBase() + '/login/logout';

-    //for firefox and safari
+    // for firefox and safari
    logoutURL = logoutURL.replace('//', '//false:false@');
    $http.post(logoutURL).error(function() {
-      //force authcBasic (if configured) to logout
+      // force authcBasic (if configured) to logout
      $http.post(logoutURL).error(function() {
        $rootScope.userName = '';
        $rootScope.ticket.principal = '';
--- a/zeppelin-web/src/components/ngenter/ngenter.directive.test.js
+++ b/zeppelin-web/src/components/ngenter/ngenter.directive.test.js
@ -16,10 +16,10 @@ describe('Directive: ngEnter', function() {
    expect(element.text()).toBeDefined();
  }));

-  //Test the rest of function in ngEnter
+  // Test the rest of function in ngEnter
  /*  it('should make hidden element visible', inject(function ($compile) {
      element = angular.element('<ng-enter></ng-enter>');
      element = $compile(element)(scope);
      expect(element.text()).toBe('this is the ngEnter directive');
-    }));*/
+    })); */
 });
--- a/zeppelin-web/src/components/noteName-create/notename.controller.js
+++ b/zeppelin-web/src/components/noteName-create/notename.controller.js
@ -99,7 +99,7 @@ function NotenameCtrl($scope, noteListDataFactory, $routeParams, websocketMsgSrv
  $scope.$on('interpreterSettings', function(event, data) {
    $scope.interpreterSettings = data.interpreterSettings;

-    //initialize default interpreter with Spark interpreter
+    // initialize default interpreter with Spark interpreter
    $scope.note.defaultInterpreter = data.interpreterSettings[0];
  });

--- a/zeppelin-web/src/components/noteName-import/notenameImport.controller.js
+++ b/zeppelin-web/src/components/noteName-import/notenameImport.controller.js
@ -119,7 +119,7 @@ function NoteImportCtrl($scope, $timeout, websocketMsgSrv) {
        result.name = $scope.note.noteImportName;
      }
      websocketMsgSrv.importNote(result);
-      //angular.element('#noteImportModal').modal('hide');
+      // angular.element('#noteImportModal').modal('hide');
    } else {
      $scope.note.errorText = 'Invalid JSON';
    }
--- a/zeppelin-web/src/components/saveAs/saveAs.service.js
+++ b/zeppelin-web/src/components/saveAs/saveAs.service.js
@ -31,8 +31,8 @@ function saveAsService(browserDetectService) {
      frameSaveAs.document.execCommand('SaveAs', false, filename + '.' + extension);
      var t2 = Date.now();

-      //This means, this version of IE dosen't support auto download of a file with extension provided in param
-      //falling back to ".txt"
+      // This means, this version of IE dosen't support auto download of a file with extension provided in param
+      // falling back to ".txt"
      if (t1 === t2) {
        frameSaveAs.document.execCommand('SaveAs', true, filename + '.txt');
      }
--- a/zeppelin-web/src/components/searchService/search.service.js
+++ b/zeppelin-web/src/components/searchService/search.service.js
@ -20,7 +20,7 @@ function searchService($resource, baseUrlSrv) {
  this.search = function(term) {
    this.searchTerm = term.q;
    console.log('Searching for: %o', term.q);
-    if (!term.q) { //TODO(bzz): empty string check
+    if (!term.q) { // TODO(bzz): empty string check
      return;
    }
    var encQuery = window.encodeURIComponent(term.q);