Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Commit aa172b7

Browse files
Splaktarandrewseguin
Splaktar
authored and
andrewseguin
committed
style(eslint): enable array-bracket-spacing rule (#11587)
1 parent c68e7f0 commit aa172b7

File tree

21 files changed

+42
-42
lines changed

21 files changed

+42
-42
lines changed

‎.eslintrc.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"rules": {
77
"accessor-pairs": "error",
88
"array-bracket-newline": "off",
9-
"array-bracket-spacing": "off",
9+
"array-bracket-spacing": ["error", "never"],
1010
"array-callback-return": "error",
1111
"array-element-newline": "off",
1212
"arrow-body-style": "error",

‎config/ngModuleData.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ function buildScanner(pattern) {
3636
return match ? {
3737
name : moduleName || '',
3838
module : moduleName || '',
39-
dependencies : dependencies || [ ]
39+
dependencies : dependencies || []
4040
} : null;
4141
};
4242
}

‎docs/app/js/app.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
angular.module('docsApp', [ 'angularytics', 'ngRoute', 'ngMessages', 'ngMaterial' ], [
1+
angular.module('docsApp', ['angularytics', 'ngRoute', 'ngMessages', 'ngMaterial'], [
22
'SERVICES',
33
'COMPONENTS',
44
'DEMOS',
@@ -361,7 +361,7 @@ function(SERVICES, COMPONENTS, DEMOS, PAGES, $location, $rootScope, $http, $wind
361361
response = response.data;
362362
var versionId = getVersionIdFromPath();
363363
var head = { type: 'version', url: '/HEAD', id: 'head', name: 'HEAD (master)', github: '' };
364-
var commonVersions = versionId === 'head' ? [] : [ head ];
364+
var commonVersions = versionId === 'head' ? [] : [head];
365365
var knownVersions = getAllVersions();
366366
var listVersions = knownVersions.filter(removeCurrentVersion);
367367
var currentVersion = getCurrentVersion() || {name: 'local'};
@@ -370,11 +370,11 @@ function(SERVICES, COMPONENTS, DEMOS, PAGES, $location, $rootScope, $http, $wind
370370
name: 'Documentation Version',
371371
type: 'heading',
372372
className: 'version-picker',
373-
children: [ {
373+
children: [{
374374
name: currentVersion.name,
375375
type: 'toggle',
376376
pages: commonVersions.concat(listVersions)
377-
} ]
377+
}]
378378
});
379379
function removeCurrentVersion (version) {
380380
switch (versionId) {

‎docs/config/processors/buildConfig.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ module.exports = function buildConfigProcessor() {
1717
buildConfig: buildConfig
1818
});
1919

20-
return q.all([ getSHA(), getCommitDate() ])
21-
.then(function(){
20+
return q.all([getSHA(), getCommitDate()])
21+
.then(function() {
2222
return docs;
2323
});
2424
}

‎gulp/tasks/build-all-modules.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const util = require('../util');
88

99
exports.task = function() {
1010
const isRelease = process.argv.indexOf('--release') !== -1;
11-
return gulp.src(['src/core/', 'src/components/*' ])
11+
return gulp.src(['src/core/', 'src/components/*'])
1212
.pipe(through2.obj(function(folder, enc, next) {
1313
const moduleId = folder.path.indexOf('components') > -1
1414
? 'material.components.' + path.basename(folder.path)

‎gulp/tasks/ddescribe-iit.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function checkFile(fileContents, disallowed) {
4141

4242
exports.task = function() {
4343
let failures = void 0;
44-
return gulp.src(['src/**/*.spec.js', 'test/**/*-spec.js' ])
44+
return gulp.src(['src/**/*.spec.js', 'test/**/*-spec.js'])
4545
.pipe(through2.obj(function(file, enc, next) {
4646
const errors = checkFile(file.contents.toString(), kDisallowedFunctions);
4747
if (errors) {

‎release.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
const child_process = require('child_process');
1111
const pkg = require('./package.json');
1212
let oldVersion = pkg.version;
13-
const abortCmds = [ 'git reset --hard', 'git checkout staging', 'rm abort push' ];
14-
const pushCmds = [ 'rm abort push' ];
13+
const abortCmds = ['git reset --hard', 'git checkout staging', 'rm abort push'];
14+
const pushCmds = ['rm abort push'];
1515
const cleanupCmds = [];
1616
const defaultOptions = { encoding: 'utf-8' };
1717
const origin = '[email protected]:angular/material.git';
@@ -134,8 +134,8 @@
134134

135135
function getVersionOptions (version) {
136136
return version.match(/-rc\.?\d+$/)
137-
? [ increment(version, 'rc'), increment(version, 'minor') ]
138-
: [ increment(version, 'patch'), addRC(increment(version, 'minor')) ];
137+
? [increment(version, 'rc'), increment(version, 'minor')]
138+
: [increment(version, 'patch'), addRC(increment(version, 'minor'))];
139139

140140
function increment (versionString, type) {
141141
const version = parseVersion(versionString);
@@ -282,7 +282,7 @@
282282

283283
// update firebase.json file
284284
updateFirebaseJson();
285-
exec([ 'git commit --amend --no-edit -a' ], options);
285+
exec(['git commit --amend --no-edit -a'], options);
286286
done();
287287

288288
// add steps to push script

‎src/components/button/button.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
* Button
77
*/
88
angular
9-
.module('material.components.button', [ 'material.core' ])
9+
.module('material.components.button', ['material.core'])
1010
.directive('mdButton', MdButtonDirective)
1111
.directive('a', MdAnchorDirective);
1212

‎src/components/gridList/demoDynamicTiles/script.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ angular
1010
});
1111

1212
function buildGridModel(tileTmpl){
13-
var it, results = [ ];
13+
var it, results = [];
1414

1515
for (var j=0; j<11; j++) {
1616

‎src/components/menu/menu.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -371,7 +371,7 @@ describe('material.components.menu', function() {
371371
' <md-button ng-click="doSomething($event)"></md-button>' +
372372
' </md-menu-item>' +
373373
' </md-menu-content>' +
374-
'</md-menu>', [ buttonAttrs || 'ng-click="$mdMenu.open($event)"' ]);
374+
'</md-menu>', [buttonAttrs || 'ng-click="$mdMenu.open($event)"']);
375375

376376
inject(function($compile, $rootScope) {
377377
$rootScope.doSomething = function($event) {

‎src/components/progressLinear/demoBasicUsage/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,13 @@ angular.module('progressLinearDemo1', ['ngMaterial'])
99
self.determinateValue = 30;
1010
self.determinateValue2 = 30;
1111

12-
self.showList = [ ];
12+
self.showList = [];
1313

1414
/**
1515
* Turn off or on the 5 themed loaders
1616
*/
1717
self.toggleActivation = function() {
18-
if (!self.activated) self.showList = [ ];
18+
if (!self.activated) self.showList = [];
1919
if (self.activated) {
2020
j = counter = 0;
2121
self.determinateValue = 30;

‎src/components/progressLinear/progress-linear.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ function MdProgressLinearDirective($mdTheming, $mdUtil, $log) {
181181
function animateIndicator(target, value) {
182182
if (isDisabled || !mode()) return;
183183

184-
var to = $mdUtil.supplant("translateX({0}%) scale({1},1)", [ (value-100)/2, value/100 ]);
184+
var to = $mdUtil.supplant("translateX({0}%) scale({1},1)", [(value-100)/2, value/100]);
185185
var styles = toVendorCSS({ transform : to });
186186
angular.element(target).css(styles);
187187
}

‎src/components/select/select.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ describe('<md-select>', function() {
154154
$rootScope.onChange = function() { changed = true; };
155155
$rootScope.val = { id: 1, name: 'Bob' };
156156

157-
var opts = [ { id: 1, name: 'Bob' }, { id: 2, name: 'Alice' } ];
157+
var opts = [{ id: 1, name: 'Bob' }, { id: 2, name: 'Alice' }];
158158
var select = setupSelect('ng-model="$root.val" ng-change="onChange()" ng-model-options="{trackBy: \'$value.id\'}"', opts);
159159
expect(changed).toBe(false);
160160

‎src/components/toast/toast.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@ function MdToastProvider($$interimElementProvider) {
337337
.addPreset('simple', {
338338
argOption: 'textContent',
339339
methods: ['textContent', 'content', 'action', 'actionKey', 'actionHint', 'highlightAction',
340-
'highlightClass', 'theme', 'parent', 'dismissHint' ],
340+
'highlightClass', 'theme', 'parent', 'dismissHint'],
341341
options: /* @ngInject */ function($mdToast, $mdTheming) {
342342
return {
343343
template:

‎src/core/services/gesture/gesture.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ var isInitialized = false;
2424
* AngularJS Material Gesture handling for touch devices. This module replaced the usage of the hammerjs library.
2525
*/
2626
angular
27-
.module('material.core.gestures', [ ])
27+
.module('material.core.gestures', [])
2828
.provider('$mdGesture', MdGestureProvider)
2929
.factory('$$MdGestureHandler', MdGestureHandler)
3030
.run(attachToDocument);

‎src/core/services/interimElement/interimElement.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ describe('$$interimElement service', function() {
1616
expect(interimTest.show).toBeOfType('function');
1717

1818
var builder = interimTest.build();
19-
[ 'controller', 'controllerAs', 'onRemove', 'onShow', 'resolve',
19+
['controller', 'controllerAs', 'onRemove', 'onShow', 'resolve',
2020
'template', 'templateUrl', 'themable', 'transformTemplate', 'parent'
2121
].forEach(function(methodName) {
2222
expect(builder[methodName]).toBeOfType('function');

‎src/core/services/layout/layout.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
var SUFFIXES = /(-gt)?-(sm|md|lg|print)/g;
77
var WHITESPACE = /\s+/g;
88

9-
var FLEX_OPTIONS = ['grow', 'initial', 'auto', 'none', 'noshrink', 'nogrow' ];
9+
var FLEX_OPTIONS = ['grow', 'initial', 'auto', 'none', 'noshrink', 'nogrow'];
1010
var LAYOUT_OPTIONS = ['row', 'column'];
11-
var ALIGNMENT_MAIN_AXIS= [ "", "start", "center", "end", "stretch", "space-around", "space-between" ];
12-
var ALIGNMENT_CROSS_AXIS= [ "", "start", "center", "end", "stretch" ];
11+
var ALIGNMENT_MAIN_AXIS= ["", "start", "center", "end", "stretch", "space-around", "space-between"];
12+
var ALIGNMENT_CROSS_AXIS= ["", "start", "center", "end", "stretch"];
1313

1414
var config = {
1515
/**
@@ -77,9 +77,9 @@
7777
var SPECIAL_CHARS_REGEXP = /([:\-_]+(.))/g;
7878

7979
// NOTE: these are also defined in constants::MEDIA_PRIORITY and constants::MEDIA
80-
var BREAKPOINTS = [ "", "xs", "gt-xs", "sm", "gt-sm", "md", "gt-md", "lg", "gt-lg", "xl", "print" ];
81-
var API_WITH_VALUES = [ "layout", "flex", "flex-order", "flex-offset", "layout-align" ];
82-
var API_NO_VALUES = [ "show", "hide", "layout-padding", "layout-margin" ];
80+
var BREAKPOINTS = ["", "xs", "gt-xs", "sm", "gt-sm", "md", "gt-md", "lg", "gt-lg", "xl", "print"];
81+
var API_WITH_VALUES = ["layout", "flex", "flex-order", "flex-offset", "layout-align"];
82+
var API_NO_VALUES = ["show", "hide", "layout-padding", "layout-margin"];
8383

8484

8585
// Build directive registration functions for the standard Layout API... for all breakpoints.
@@ -213,7 +213,7 @@
213213
* finish processing. Eliminates flicker with Material.Layouts
214214
*/
215215
function buildCloakInterceptor(className) {
216-
return [ '$timeout', function($timeout){
216+
return ['$timeout', function($timeout){
217217
return {
218218
restrict : 'A',
219219
priority : -10, // run after normal ng-cloak
@@ -510,7 +510,7 @@
510510
values = attrValue.toLowerCase().trim().replace(WHITESPACE, "-").split("-");
511511
if (values.length && (values[0] === "space")) {
512512
// for main-axis values of "space-around" or "space-between"
513-
values = [ values[0]+"-"+values[1],values[2] ];
513+
values = [values[0]+"-"+values[1],values[2]];
514514
}
515515

516516
if (values.length > 0) axis.main = values[0] || axis.main;

‎src/core/services/registry/componentRegistry.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
function ComponentRegistry($log, $q) {
2020

2121
var self;
22-
var instances = [ ];
22+
var instances = [];
2323
var pendings = { };
2424

2525
return self = {

‎src/core/util/iterator.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
}
2828

2929
reloop = !!reloop;
30-
var _items = items || [ ];
30+
var _items = items || [];
3131

3232
// Published API
3333
return {

‎src/core/util/iterator.spec.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ describe('iterator', function() {
66
var list, iter;
77

88
beforeEach(inject(function ($mdUtil) {
9-
list = [ 13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99 ];
9+
list = [13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99];
1010
iter = $mdUtil.iterator(list);
1111
}));
1212

@@ -78,7 +78,7 @@ describe('iterator', function() {
7878
var list, iter;
7979

8080
beforeEach(inject(function ($mdUtil) {
81-
list = [ 13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99 ];
81+
list = [13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99];
8282
iter = $mdUtil.iterator(list);
8383
}));
8484

@@ -133,7 +133,7 @@ describe('iterator', function() {
133133
var list, iter;
134134

135135
beforeEach(inject(function ($mdUtil) {
136-
list = [ 13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99 ];
136+
list = [13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99];
137137
iter = $mdUtil.iterator(list);
138138
}));
139139

@@ -175,7 +175,7 @@ describe('iterator', function() {
175175
expect(iter.last()).toBe('Max');
176176
expect(iter.first()).toBe(13);
177177

178-
iter = $mdUtil.iterator([ 2, 5 ]);
178+
iter = $mdUtil.iterator([2, 5]);
179179
iter.remove(2);
180180
expect(iter.last()).toBe(iter.first());
181181

@@ -211,7 +211,7 @@ describe('iterator', function() {
211211
iter.remove(13);
212212
expect(iter.hasPrevious(iter.first())).toBe(false);
213213

214-
iter = $mdUtil.iterator(list = [ 2, 3 ]);
214+
iter = $mdUtil.iterator(list = [2, 3]);
215215
expect(iter.hasPrevious(iter.last())).toBe(true);
216216
iter.remove(2);
217217
expect(iter.hasPrevious(iter.last())).toBe(false);
@@ -261,7 +261,7 @@ describe('iterator', function() {
261261
var validate = function(item) { return (item !== 14) && (item !== 'Andrew'); };
262262

263263
beforeEach(inject(function ($mdUtil) {
264-
list = [ 13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99 ];
264+
list = [13, 14, 'Marcy', 15, 'Andrew', 16, 21, 'Adam', 37, 'Max', 99];
265265
iter = $mdUtil.iterator(list);
266266
}));
267267

‎src/core/util/util.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,7 @@ function UtilFactory($document, $timeout, $compile, $rootScope, $$mdAnimate, $in
466466
*/
467467
valueOnUse : function (scope, key, getter) {
468468
var value = null, args = Array.prototype.slice.call(arguments);
469-
var params = (args.length > 3) ? args.slice(3) : [ ];
469+
var params = (args.length > 3) ? args.slice(3) : [];
470470

471471
Object.defineProperty(scope, key, {
472472
get: function () {

0 commit comments

Comments
 (0)