Require trailing commas on multi-line array/object literals

Add an ESLint rule to require this and fix up non-conforming code using
ESLint's --fix tool.

.eslintrc

@@ -13,6 +13,7 @@
   "rules": {
     "array-callback-return": "error",
     "block-scoped-var": "error",
+    "comma-dangle": ["error", "always-multiline"],
     "consistent-return": "error",
     "curly": "error",
     "dot-notation": "error",

gulpfile.js

@@ -293,7 +293,7 @@ function runKarma(baseConfig, opts, done) {
     client: {
       mocha: {
         grep: taskArgs.grep,
-      }
+      },
     },
   };
 

h/static/scripts/annotation-mapper.js

@@ -48,7 +48,7 @@ function annotationMapper($rootScope, annotationUI, store) {
 
   function deleteAnnotation(annotation) {
     return store.annotation.delete({
-      id: annotation.id
+      id: annotation.id,
     }).then(function () {
       $rootScope.$emit(events.ANNOTATION_DELETED, annotation);
       return annotation;
@@ -59,7 +59,7 @@ function annotationMapper($rootScope, annotationUI, store) {
     loadAnnotations: loadAnnotations,
     unloadAnnotations: unloadAnnotations,
     createAnnotation: createAnnotation,
-    deleteAnnotation: deleteAnnotation
+    deleteAnnotation: deleteAnnotation,
   };
 }
 

h/static/scripts/annotation-ui-sync.js

@@ -55,7 +55,7 @@ function AnnotationUISync($rootScope, $window, annotationUI, bridge) {
         annotationUI.setShowHighlights(state);
         bridge.call('setVisibleHighlights', state);
       }
-    }
+    },
   };
 
   for (var channel in channelListeners) {

h/static/scripts/annotator/plugin/pdf-metadata.js

@@ -53,7 +53,7 @@ PDFMetadata.prototype.getMetadata = function () {
     }
 
     var link = [
-      {href: fingerprintToURN(app.documentFingerprint)}
+      {href: fingerprintToURN(app.documentFingerprint)},
     ];
 
     var url = getPDFURL(app);

h/static/scripts/annotator/plugin/test/pdf-metadata-test.js

@@ -39,7 +39,7 @@ describe('pdf-metadata', function () {
       metadata: {
         metadata: {
           'dc:title': 'fakeTitle',
-        }
+        },
       },
       url: 'http://fake.com',
     };

h/static/scripts/app.js

@@ -52,24 +52,24 @@ function configureRoutes($routeProvider) {
       controller: 'AnnotationViewerController',
       template: VIEWER_TEMPLATE,
       reloadOnSearch: false,
-      resolve: resolve
+      resolve: resolve,
     });
   $routeProvider.when('/viewer',
     {
       controller: 'WidgetController',
       template: VIEWER_TEMPLATE,
       reloadOnSearch: false,
-      resolve: resolve
+      resolve: resolve,
     });
   $routeProvider.when('/stream',
     {
       controller: 'StreamController',
       template: VIEWER_TEMPLATE,
       reloadOnSearch: false,
-      resolve: resolve
+      resolve: resolve,
     });
   return $routeProvider.otherwise({
-    redirectTo: '/viewer'
+    redirectTo: '/viewer',
   });
 }
 
@@ -115,7 +115,7 @@ module.exports = angular.module('h', [
   ['ui.bootstrap', require('./vendor/ui-bootstrap-custom-tpls-0.13.4')][0],
 
   // Local addons
-  'ngRaven'
+  'ngRaven',
 ])
 
   .controller('AppController', require('./app-controller'))

h/static/scripts/auth.js

@@ -34,7 +34,7 @@ function fetchToken($http, session, settings) {
     skipAuthorization: true,
     transformRequest: function (data) {
       return queryString.stringify(data);
-    }
+    },
   };
   return $http.get(tokenUrl, config).then(function (response) {
     return response.data;

h/static/scripts/directive/annotation-share-dialog.js

@@ -71,6 +71,6 @@ module.exports = function () {
       isPrivate: '<',
       isOpen: '<',
       onClose: '&',
-    }
+    },
   };
 };

h/static/scripts/directive/annotation.js

@@ -441,7 +441,7 @@ function AnnotationController(
     drafts.update(vm.annotation, {
       tags: vm.state().tags,
       text: vm.state().text,
-      isPrivate: privacy === 'private'
+      isPrivate: privacy === 'private',
     });
   };
 
@@ -544,7 +544,7 @@ function annotation() {
       showDocumentInfo: '<',
       onReplyCountClick: '&',
       replyCount: '<',
-      isCollapsed: '<'
+      isCollapsed: '<',
     },
     template: require('../../../templates/client/annotation.html'),
   };
@@ -560,5 +560,5 @@ module.exports = {
 
   // These are meant to be the public API of this module.
   directive: annotation,
-  Controller: AnnotationController
+  Controller: AnnotationController,
 };

h/static/scripts/directive/group-list.js

@@ -35,7 +35,7 @@ function groupList( $window, groups, settings) {
     },
     restrict: 'E',
     scope: {
-      auth: '<'
+      auth: '<',
     },
     template: require('../../../templates/client/group_list.html'),
   };
@@ -43,5 +43,5 @@ function groupList( $window, groups, settings) {
 
 module.exports = {
   directive: groupList,
-  Controller: GroupListController
+  Controller: GroupListController,
 };

h/static/scripts/directive/h-autofocus.js

@@ -18,6 +18,6 @@ module.exports = function() {
     restrict: 'A',
     link: function($scope, $element) {
       $element[0].focus();
-    }
+    },
   };
 };

h/static/scripts/directive/help-panel.js

@@ -35,6 +35,6 @@ module.exports = function () {
     scope: {
       auth: '<',
       onClose: '&',
-    }
+    },
   };
 };

h/static/scripts/directive/markdown.js

@@ -116,7 +116,7 @@ module.exports = function($sanitize) {
         var shortcuts = {
           66: scope.insertBold,
           73: scope.insertItalic,
-          75: scope.insertLink
+          75: scope.insertLink,
         };
 
         var shortcut = shortcuts[e.keyCode];

h/static/scripts/directive/publish-annotation-btn.js

@@ -34,7 +34,7 @@ module.exports = function () {
       isShared: '<',
       onCancel: '&',
       onSave: '&',
-      onSetPrivacy: '&'
+      onSetPrivacy: '&',
     },
     template: require('../../../templates/client/publish_annotation_btn.html'),
   };

h/static/scripts/directive/sidebar-tutorial.js

@@ -36,5 +36,5 @@ module.exports = {
       template: require('../../../templates/client/sidebar_tutorial.html'),
     };
   },
-  Controller: SidebarTutorialController
+  Controller: SidebarTutorialController,
 };

h/static/scripts/directive/spinner.js

@@ -7,6 +7,6 @@ module.exports = ['$animate', function($animate) {
       $animate.enabled(false, elem);
     },
     restrict: 'C',
-    template: '<span><span></span></span>'
+    template: '<span><span></span></span>',
   };
 }];

h/static/scripts/directive/test/annotation-share-dialog-test.js

@@ -25,7 +25,7 @@ describe('annotationShareDialog', function () {
   describe('The annotation share dialog', function () {
     it('has class is-open set when it is open', function () {
       element = util.createDirective(document, 'annotationShareDialog', {
-        isOpen: true
+        isOpen: true,
       });
 
       assert.isOk(element.find('.annotation-share-dialog').hasClass('is-open'));
@@ -33,7 +33,7 @@ describe('annotationShareDialog', function () {
 
     it('does not have class is-open set when it is not open', function () {
       element = util.createDirective(document, 'annotationShareDialog', {
-        isOpen: false
+        isOpen: false,
       });
 
       assert.isNotOk(element.find('.annotation-share-dialog').hasClass('is-open'));
@@ -51,7 +51,7 @@ describe('annotationShareDialog', function () {
           group: {
             name: 'Public',
             type: 'public',
-            public: true
+            public: true,
           },
           uri: 'fakeURI',
           isPrivate: false,
@@ -101,9 +101,9 @@ describe('annotationShareDialog', function () {
     it('is available to a group', function () {
       element = util.createDirective(document, 'annotationShareDialog', {
         group: {
-          public: false
+          public: false,
         },
-        isPrivate: false
+        isPrivate: false,
       });
 
       var actualMessage = element.find('.annotation-share-dialog-msg').text();
@@ -116,7 +116,7 @@ describe('annotationShareDialog', function () {
 
     it('is private', function () {
       element = util.createDirective(document, 'annotationShareDialog', {
-        isPrivate: true
+        isPrivate: true,
       });
 
       var actualMessage = element.find('.annotation-share-dialog-msg').text();

h/static/scripts/directive/test/annotation-test.js

@@ -32,7 +32,7 @@ function annotationDirective() {
       domainAndTitle: function (annot) { // eslint-disable-line no-unused-vars
         return fakeDocumentMeta;
       },
-    }
+    },
   });
 
   return annotation.directive;
@@ -125,10 +125,10 @@ describe('annotation', function() {
             read: ['acct:bill@localhost'],
             update: ['acct:bill@localhost'],
             destroy: ['acct:bill@localhost'],
-            admin: ['acct:bill@localhost']
-          }
+            admin: ['acct:bill@localhost'],
+          },
         }),
-        deleteAnnotation: sandbox.stub()
+        deleteAnnotation: sandbox.stub(),
       };
 
       var fakeAnnotationUI = {};
@@ -136,7 +136,7 @@ describe('annotation', function() {
       fakeDrafts = {
         update: sandbox.stub(),
         remove: sandbox.stub(),
-        get: sandbox.stub()
+        get: sandbox.stub(),
       };
 
       var fakeFeatures = {
@@ -152,21 +152,21 @@ describe('annotation', function() {
         isPrivate: sandbox.stub().returns(false),
         permits: sandbox.stub().returns(true),
         shared: sandbox.stub().returns({
-          read: ['everybody']
+          read: ['everybody'],
         }),
         'private': sandbox.stub().returns({
-          read: ['justme']
+          read: ['justme'],
         }),
         'default': sandbox.stub().returns({
-          read: ['default']
+          read: ['default'],
         }),
-        setDefault: sandbox.stub()
+        setDefault: sandbox.stub(),
       };
 
       fakeSession = {
         state: {
-          userid: 'acct:bill@localhost'
-        }
+          userid: 'acct:bill@localhost',
+        },
       };
 
       var fakeSettings = {
@@ -177,7 +177,7 @@ describe('annotation', function() {
         focused: function() {
           return {};
         },
-        get: function() {}
+        get: function() {},
       };
 
       fakeStore = {
@@ -270,8 +270,8 @@ describe('annotation', function() {
             read: ['foo'],
             update: ['bar'],
             'delete': ['gar'],
-            admin: ['har']
-          }
+            admin: ['har'],
+          },
         };
         var originalPermissions = JSON.parse(JSON.stringify(
           annotation.permissions));
@@ -454,7 +454,7 @@ describe('annotation', function() {
         createDirective(annotation);
         $scope.$digest();
         assert.deepEqual(annotation.permissions, {
-          read: ['justme']
+          read: ['justme'],
         });
       });
     });
@@ -468,7 +468,7 @@ describe('annotation', function() {
           read: ['acct:joe@localhost'],
           update: ['acct:joe@localhost'],
           destroy: ['acct:joe@localhost'],
-          admin: ['acct:joe@localhost']
+          admin: ['acct:joe@localhost'],
         };
       });
 
@@ -476,7 +476,7 @@ describe('annotation', function() {
         var controller = createDirective(annotation).controller;
         var reply = sinon.match({
           references: [annotation.id],
-          uri: annotation.uri
+          uri: annotation.uri,
         });
         controller.reply();
         assert.calledWith(fakeAnnotationMapper.createAnnotation, reply);
@@ -488,7 +488,7 @@ describe('annotation', function() {
         var reply = sinon.match({
           references: [annotation.id],
           permissions: perms,
-          uri: annotation.uri
+          uri: annotation.uri,
         });
         fakePermissions.isShared.returns(true);
         fakePermissions.shared.returns(perms);
@@ -582,10 +582,10 @@ describe('annotation', function() {
           {
             selector: [
               {
-                type: 'TextQuoteSelector'
-              }
-            ]
-          }
+                type: 'TextQuoteSelector',
+              },
+            ],
+          },
         ];
         var controller = createDirective(annotation).controller;
 
@@ -632,7 +632,7 @@ describe('annotation', function() {
           var controller = createDirective().controller;
           sandbox.stub($window, 'confirm').returns(true);
           fakeAnnotationMapper.deleteAnnotation.returns($q.reject({
-            status: 0
+            status: 0,
           }));
           controller.delete().then(function() {
             assert.calledWith(fakeFlash.error,
@@ -649,7 +649,7 @@ describe('annotation', function() {
         fakeAnnotationMapper.deleteAnnotation.returns($q.reject({
           status: 500,
           statusText: 'Server Error',
-          data: {}
+          data: {},
         }));
         controller.delete().then(function() {
           assert.calledWith(fakeFlash.error,
@@ -708,7 +708,7 @@ describe('annotation', function() {
       it('flashes a generic error if the server can\'t be reached', function() {
         var controller = createController();
         fakeStore.annotation.create = sinon.stub().returns(Promise.reject({
-          status: 0
+          status: 0,
         }));
         return controller.save().then(function() {
           assert.calledWith(fakeFlash.error,
@@ -721,7 +721,7 @@ describe('annotation', function() {
         fakeStore.annotation.create = sinon.stub().returns(Promise.reject({
           status: 500,
           statusText: 'Server Error',
-          data: {}
+          data: {},
         }));
         return controller.save().then(function() {
           assert.calledWith(fakeFlash.error,
@@ -782,7 +782,7 @@ describe('annotation', function() {
       it('flashes a generic error if the server cannot be reached', function () {
         var controller = createController();
         fakeStore.annotation.update = sinon.stub().returns(Promise.reject({
-          status: -1
+          status: -1,
         }));
         return controller.save().then(function() {
           assert.calledWith(fakeFlash.error,
@@ -795,7 +795,7 @@ describe('annotation', function() {
         fakeStore.annotation.update = sinon.stub().returns(Promise.reject({
           status: 500,
           statusText: 'Server Error',
-          data: {}
+          data: {},
         }));
         return controller.save().then(function() {
           assert.calledWith(fakeFlash.error,
@@ -815,7 +815,7 @@ describe('annotation', function() {
       it('starts editing immediately if there is a draft', function() {
         fakeDrafts.get.returns({
           tags: ['unsaved'],
-          text: 'unsaved-text'
+          text: 'unsaved-text',
         });
         var controller = createDirective().controller;
         assert.isTrue(controller.editing());
@@ -824,7 +824,7 @@ describe('annotation', function() {
       it('uses the text and tags from the draft if present', function() {
         fakeDrafts.get.returns({
           tags: ['unsaved-tag'],
-          text: 'unsaved-text'
+          text: 'unsaved-text',
         });
         var controller = createDirective().controller;
         assert.deepEqual(controller.state().tags, ['unsaved-tag']);
@@ -947,7 +947,7 @@ describe('annotation', function() {
       it('displays links to tags on the stream', function () {
         var directive = createDirective({
           id: '1234',
-          tags: ['atag']
+          tags: ['atag'],
         });
         var links = [].slice.apply(directive.element[0].querySelectorAll('a'));
         var tagLinks = links.filter(function (link) {
@@ -964,7 +964,7 @@ describe('annotation', function() {
         var annotation = Object.assign({}, fixtures.defaultAnnotation(), {
           links: {
             html: 'https://test.hypothes.is/a/deadbeef',
-            incontext: 'https://hpt.is/deadbeef'
+            incontext: 'https://hpt.is/deadbeef',
           },
         });
         var controller = createDirective(annotation).controller;
@@ -985,7 +985,7 @@ describe('annotation', function() {
         var annotation = Object.assign({}, fixtures.defaultAnnotation(), {
           links: {
             html: 'https://test.hypothes.is/a/deadbeef',
-            incontext: 'https://hpt.is/deadbeef'
+            incontext: 'https://hpt.is/deadbeef',
           },
         });
         var controller = createDirective(annotation).controller;

h/static/scripts/directive/test/annotation-thread-test.js

@@ -57,8 +57,8 @@ describe('annotationThread', function () {
         id: '1',
         annotation: {id: '1'},
         visible: false,
-        children: []
-      }
+        children: [],
+      },
     });
     var pageObject = new PageObject(element);
     assert.equal(pageObject.annotations().length, 1);
@@ -78,7 +78,7 @@ describe('annotationThread', function () {
           visible: true,
         }],
         collapsed: false,
-      }
+      },
     });
     var pageObject = new PageObject(element);
     assert.isFalse(pageObject.isHidden(pageObject.replyList()));
@@ -97,7 +97,7 @@ describe('annotationThread', function () {
           visible: true,
         }],
         collapsed: true,
-      }
+      },
     });
     var pageObject = new PageObject(element);
     assert.isTrue(pageObject.isHidden(pageObject.replyList()));
@@ -121,7 +121,7 @@ describe('annotationThread', function () {
           visible: true,
         }],
         collapsed: false,
-      }
+      },
     });
     var pageObject = new PageObject(element);
     assert.equal(pageObject.visibleReplies().length, 1);
@@ -140,7 +140,7 @@ describe('annotationThread', function () {
         onChangeCollapsed: {
           args: ['id', 'collapsed'],
           callback: onChangeCollapsed,
-        }
+        },
       });
       element.ctrl.toggleCollapsed();
       assert.calledWith(onChangeCollapsed, '123', false);

h/static/scripts/directive/test/excerpt-test.js

@@ -219,7 +219,7 @@ describe('excerpt directive', function () {
         onCollapsibleChanged: {
           args: ['collapsible'],
           callback: callback,
-        }
+        },
       }, '<span></span>');
       fakeOverflowMonitor.ctrl.onOverflowChanged(true);
       assert.calledWith(callback, true);

h/static/scripts/directive/test/group-list-test.js

@@ -36,11 +36,11 @@ describe('groupList', function () {
 
     groups = [{
        id: 'public',
-       public: true
+       public: true,
     },{
        id: 'h-devs',
        name: 'Hypothesis Developers',
-       url: GROUP_LINK
+       url: GROUP_LINK,
     }];
 
     fakeGroups = {

h/static/scripts/directive/test/publish-annotation-btn-test.js

@@ -11,7 +11,7 @@ var fakeLocalStorage = {
   },
   getItem: function (key) {
     return fakeStorage[key];
-  }
+  },
 };
 
 describe('publishAnnotationBtn', function () {
@@ -34,13 +34,13 @@ describe('publishAnnotationBtn', function () {
     element = util.createDirective(document, 'publishAnnotationBtn', {
      group: {
        name: 'Public',
-       type: 'public'
+       type: 'public',
      },
      canPost: true,
      isShared: false,
      onSave: function () {},
      onSetPrivacy: function () {},
-     onCancel: function () {}
+     onCancel: function () {},
    });
   });
 
@@ -54,9 +54,9 @@ describe('publishAnnotationBtn', function () {
     element.link({
       group: {
         name: 'Research Lab',
-        type: 'group'
+        type: 'group',
       },
-      isShared: true
+      isShared: true,
     });
     var buttons = element.find('button');
     assert.equal(buttons[0].innerHTML, 'Post to Research Lab');
@@ -65,7 +65,7 @@ describe('publishAnnotationBtn', function () {
   it('should save when "Post..." is clicked', function () {
     var savedSpy = sinon.spy();
     element.link({
-      onSave: savedSpy
+      onSave: savedSpy,
     });
     assert.ok(!savedSpy.called);
     angular.element(element.find('button')[0]).click();
@@ -77,7 +77,7 @@ describe('publishAnnotationBtn', function () {
     element.link({
       // for existing annotations, the privacy should not be changed
       // unless the user makes a choice from the list
-      onSetPrivacy: privacyChangedSpy
+      onSetPrivacy: privacyChangedSpy,
     });
 
     assert.ok(!privacyChangedSpy.called);
@@ -91,14 +91,14 @@ describe('publishAnnotationBtn', function () {
 
   it('should disable post buttons when posting is not possible', function () {
     element.link({
-      canPost: false
+      canPost: false,
     });
     var disabledBtns = element.find('button[disabled]');
     assert.equal(disabledBtns.length, 1);
 
     // check that buttons are enabled when posting is possible
     element.link({
-      canPost: true
+      canPost: true,
     });
     disabledBtns = element.find('button[disabled]');
     assert.equal(disabledBtns.length, 0);
@@ -107,7 +107,7 @@ describe('publishAnnotationBtn', function () {
   it('should revert changes when cancel is clicked', function () {
     var cancelSpy = sinon.spy();
     element.link({
-      onCancel: cancelSpy
+      onCancel: cancelSpy,
     });
     var cancelBtn = element.find('.publish-annotation-cancel-btn');
     assert.equal(cancelBtn.length, 1);

h/static/scripts/directive/test/search-status-bar-test.js

@@ -18,7 +18,7 @@ describe('searchStatusBar', function () {
     it('should display the filter count', function () {
       var elem = util.createDirective(document, 'searchStatusBar', {
         filterActive: true,
-        filterMatchCount: 5
+        filterMatchCount: 5,
       });
       assert.include(elem[0].textContent, "5 search results");
     });

h/static/scripts/directive/test/sidebar-tutorial-test.js

@@ -9,9 +9,9 @@ describe('SidebarTutorialController', function () {
       var session = {
         state: {
           preferences: {
-            show_sidebar_tutorial: true
-          }
-        }
+            show_sidebar_tutorial: true,
+          },
+        },
       };
       var controller = new Controller(session);
 
@@ -24,9 +24,9 @@ describe('SidebarTutorialController', function () {
       var session = {
         state: {
           preferences: {
-            show_sidebar_tutorial: false
-          }
-        }
+            show_sidebar_tutorial: false,
+          },
+        },
       };
       var controller = new Controller(session);
 

h/static/scripts/directive/test/sort-dropdown-test.js

@@ -22,7 +22,7 @@ describe('sortDropdown', function () {
       onChangeSortKey: {
         args: ['sortKey'],
         callback: changeSpy,
-      }
+      },
     });
     var links = elem.find('li');
     angular.element(links[0]).click();

h/static/scripts/directive/test/tag-editor-test.js

@@ -25,7 +25,7 @@ describe('tagEditor', function () {
 
   it('converts tags to the form expected by ng-tags-input', function () {
     var element = util.createDirective(document, 'tag-editor', {
-      tags: ['foo', 'bar']
+      tags: ['foo', 'bar'],
     });
     assert.deepEqual(element.ctrl.tagList, [{text: 'foo'}, {text: 'bar'}]);
   });

h/static/scripts/directive/test/window-scroll-test.js

@@ -23,7 +23,7 @@ describe('windowScroll', function () {
     html = {};
     view = {
       addEventListener: sinon.spy(),
-      removeEventListener: sinon.spy()
+      removeEventListener: sinon.spy(),
     };
 
     doc = {documentElement: html, defaultView: view};

h/static/scripts/directive/window-scroll.js

@@ -25,6 +25,6 @@ module.exports = function () {
       scope.$on('$destroy', function () {
         view.removeEventListener('scroll', onScroll);
       });
-    }
+    },
   };
 };

h/static/scripts/drafts.js

@@ -74,7 +74,7 @@ function DraftStore() {
       model: model,
       isPrivate: changes.isPrivate,
       tags: changes.tags,
-      text: changes.text
+      text: changes.text,
     };
     this.remove(model);
     this._drafts.push(newDraft);

h/static/scripts/features.js

@@ -40,7 +40,7 @@ function features($log, session) {
   }
 
   return {
-    flagEnabled: flagEnabled
+    flagEnabled: flagEnabled,
   };
 }
 

h/static/scripts/karma.config.js

@@ -12,7 +12,7 @@ module.exports = function(config) {
       'browserify',
       'mocha',
       'chai',
-      'sinon'
+      'sinon',
     ],
 
     // list of files / patterns to load in the browser
@@ -35,7 +35,7 @@ module.exports = function(config) {
       { pattern: '**/test/*-test.js', watched: false, included: true, served: true },
 
       // Integration tests
-      { pattern: '**/integration/*-test.js', watched: false, included: true, served: true }
+      { pattern: '**/integration/*-test.js', watched: false, included: true, served: true },
     ],
 
     // list of files to exclude
@@ -60,7 +60,7 @@ module.exports = function(config) {
         bundle
           .transform('coffeeify')
           .plugin('proxyquire-universal');
-      }
+      },
     },
 
     mochaReporter: {

h/static/scripts/session.js

@@ -24,7 +24,7 @@ function sessionActions(options) {
     dismiss_sidebar_tutorial: {
       method: 'POST',
       params: { path: 'dismiss_sidebar_tutorial' },
-    }
+    },
   };
 
   Object.keys(actions).forEach(function (action) {
@@ -54,7 +54,7 @@ function session($http, $resource, $rootScope, flash, raven, settings) {
   var actions = sessionActions({
     headers: headers,
     transformResponse: process,
-    withCredentials: true
+    withCredentials: true,
   });
   var endpoint = new URL('app/:path', settings.serviceUrl).href;
   var resource = $resource(endpoint, {}, actions);

h/static/scripts/test/annotation-fixtures.js

@@ -7,7 +7,7 @@ function defaultAnnotation() {
   return {
     id: 'deadbeef',
     document: {
-      title: 'A special document'
+      title: 'A special document',
     },
     target: [{source: 'source', 'selector': []}],
     uri: 'http://example.com',
@@ -26,7 +26,7 @@ function newAnnotation() {
     target: ['foo', 'bar'],
     references: [],
     text: 'Annotation text',
-    tags: ['tag_1', 'tag_2']
+    tags: ['tag_1', 'tag_2'],
   };
 }
 
@@ -38,7 +38,7 @@ function newReply() {
     target: ['foo', 'bar'],
     references: ['parent-id'],
     text: 'Annotation text',
-    tags: ['tag_1', 'tag_2']
+    tags: ['tag_1', 'tag_2'],
   };
 }
 
@@ -60,7 +60,7 @@ function newEmptyAnnotation() {
 function newHighlight() {
   return {
     id: undefined,
-    $highlight: true
+    $highlight: true,
   };
 }
 
@@ -74,7 +74,7 @@ function oldAnnotation() {
     target: ['foo', 'bar'],
     references: [],
     text: 'This is my annotation',
-    tags: ['tag_1', 'tag_2']
+    tags: ['tag_1', 'tag_2'],
   };
 }
 
@@ -88,7 +88,7 @@ function oldHighlight() {
     target: ['foo', 'bar'],
     references: [],
     text: '',
-    tags: []
+    tags: [],
   };
 }
 
@@ -101,7 +101,7 @@ function oldPageNote() {
     target: [],
     references: [],
     text: '',
-    tags: []
+    tags: [],
   };
 }
 
@@ -114,7 +114,7 @@ function oldReply() {
     target: ['foo'],
     references: ['parent_annotation_id'],
     text: '',
-    tags: []
+    tags: [],
   };
 }
 

h/static/scripts/test/annotation-mapper-test.js

@@ -86,7 +86,7 @@ describe('annotationMapper', function() {
       assert.called($rootScope.$emit);
       assert.calledWith($rootScope.$emit, events.ANNOTATION_UPDATED, {
         id: 1,
-        url: 'http://example.com'
+        url: 'http://example.com',
       });
     });
 
@@ -118,7 +118,7 @@ describe('annotationMapper', function() {
       annotationMapper.unloadAnnotations(annotations);
       assert.calledWith($rootScope.$emit, events.ANNOTATIONS_UNLOADED, [{
         id: 1,
-        url: 'http://example.com'
+        url: 'http://example.com',
       }]);
     });
   });

h/static/scripts/test/annotation-metadata-test.js

@@ -12,7 +12,7 @@ describe('annotation-metadata', function () {
       it('returns the hostname from model.uri as the domain', function() {
         var model = {
           document: {},
-          uri: 'http://example.com/'
+          uri: 'http://example.com/',
         };
 
         assert.equal(documentMetadata(model).domain, 'example.com');
@@ -22,7 +22,7 @@ describe('annotation-metadata', function () {
         it('uses model.uri as the uri', function() {
           var model = {
             document: {},
-            uri: 'http://example.com/'
+            uri: 'http://example.com/',
           };
 
           assert.equal(
@@ -35,7 +35,7 @@ describe('annotation-metadata', function () {
           var model = {
             uri: 'http://example.com/',
             document: {
-              title: ['My Document', 'My Other Document']
+              title: ['My Document', 'My Other Document'],
             },
           };
 
@@ -93,7 +93,7 @@ describe('annotation-metadata', function () {
         var model = {
           links: {
             incontext: 'https://example.com',
-          }
+          },
         };
 
         assert.equal(domainAndTitle(model).titleLink, 'https://example.com');
@@ -197,8 +197,8 @@ describe('annotation-metadata', function () {
           selector: [{
             type: 'TextPositionSelector',
             start: 100,
-          }]
-        }]
+          }],
+        }],
       }), 100);
     });
 
@@ -206,7 +206,7 @@ describe('annotation-metadata', function () {
       assert.equal(annotationMetadata.location({
         target: [{
           selector: undefined,
-        }]
+        }],
       }), Number.POSITIVE_INFINITY);
     });
   });
@@ -214,23 +214,23 @@ describe('annotation-metadata', function () {
   describe('.isPageNote', function () {
     it ('returns true for an annotation with an empty target', function () {
       assert.isTrue(annotationMetadata.isPageNote({
-        target: []
+        target: [],
       }));
     });
     it ('returns true for an annotation without selectors', function () {
       assert.isTrue(annotationMetadata.isPageNote({
-        target: [{selector: undefined}]
+        target: [{selector: undefined}],
       }));
     });
     it ('returns true for an annotation without a target', function () {
       assert.isTrue(annotationMetadata.isPageNote({
-        target: undefined
+        target: undefined,
       }));
     });
     it ('returns false for an annotation which is a reply', function () {
       assert.isFalse(annotationMetadata.isPageNote({
         target: [],
-        references: ['xyz']
+        references: ['xyz'],
       }));
     });
   });
@@ -238,7 +238,7 @@ describe('annotation-metadata', function () {
   describe ('.isAnnotation', function () {
     it ('returns true if an annotation is a top level annotation', function () {
       assert.isTrue(annotationMetadata.isAnnotation({
-        target: [{selector: []}]
+        target: [{selector: []}],
       }));
     });
     it ('returns false if an annotation has no target', function () {

h/static/scripts/test/annotation-ui-sync-test.js

@@ -36,8 +36,8 @@ describe('AnnotationUISync', function () {
       links: [
         { window: PARENT_WINDOW, channel: createChannel() },
         { window: 'ANOTHER_WINDOW', channel: createChannel() },
-        { window: 'THIRD_WINDOW', channel: createChannel() }
-      ]
+        { window: 'THIRD_WINDOW', channel: createChannel() },
+      ],
     };
 
     annotationUI = annotationUIFactory($rootScope, {});

h/static/scripts/test/annotation-ui-test.js

@@ -120,7 +120,7 @@ describe('annotationUI', function () {
     it('adds the passed annotations to the focusedAnnotationMap', function () {
       annotationUI.focusAnnotations([1, 2, 3]);
       assert.deepEqual(annotationUI.getState().focusedAnnotationMap, {
-        1: true, 2: true, 3: true
+        1: true, 2: true, 3: true,
       });
     });
 
@@ -128,7 +128,7 @@ describe('annotationUI', function () {
       annotationUI.focusAnnotations([1]);
       annotationUI.focusAnnotations([2, 3]);
       assert.deepEqual(annotationUI.getState().focusedAnnotationMap, {
-        2: true, 3: true
+        2: true, 3: true,
       });
     });
 
@@ -170,7 +170,7 @@ describe('annotationUI', function () {
     it('adds the passed annotations to the selectedAnnotationMap', function () {
       annotationUI.selectAnnotations([1, 2, 3]);
       assert.deepEqual(annotationUI.getState().selectedAnnotationMap, {
-        1: true, 2: true, 3: true
+        1: true, 2: true, 3: true,
       });
     });
 
@@ -178,7 +178,7 @@ describe('annotationUI', function () {
       annotationUI.selectAnnotations([1]);
       annotationUI.selectAnnotations([2, 3]);
       assert.deepEqual(annotationUI.getState().selectedAnnotationMap, {
-        2: true, 3: true
+        2: true, 3: true,
       });
     });
 
@@ -194,7 +194,7 @@ describe('annotationUI', function () {
       annotationUI.selectAnnotations([1, 2]);
       annotationUI.toggleSelectedAnnotations([3, 4]);
       assert.deepEqual(annotationUI.getState().selectedAnnotationMap, {
-        1: true, 2: true, 3: true, 4: true
+        1: true, 2: true, 3: true, 4: true,
       });
     });
 
@@ -216,7 +216,7 @@ describe('annotationUI', function () {
       annotationUI.selectAnnotations([1, 2, 3]);
       annotationUI.removeSelectedAnnotation(2);
       assert.deepEqual(annotationUI.getState().selectedAnnotationMap, {
-        1: true, 3: true
+        1: true, 3: true,
       });
     });
 

h/static/scripts/test/annotation-viewer-controller-test.js

@@ -16,7 +16,7 @@ function FakeStore(annots) {
         });
       }
       return Promise.resolve(result);
-    }
+    },
   };
 
   this.search = function (query) {
@@ -59,7 +59,7 @@ describe('AnnotationViewerController', function () {
       annotationUI: {
         setCollapsed: sinon.stub(),
         highlightAnnotations: sinon.stub(),
-        subscribe: sinon.stub()
+        subscribe: sinon.stub(),
       },
       rootThread: {thread: sinon.stub()},
       streamer: { setConfig: function () {} },
@@ -69,12 +69,12 @@ describe('AnnotationViewerController', function () {
           return {
             addClause: function () {
               return {
-                addClause: function () {}
+                addClause: function () {},
               };
-            }
+            },
           };
         },
-        getFilter: function () {}
+        getFilter: function () {},
       },
       annotationMapper: {
         loadAnnotations: sinon.spy(),

h/static/scripts/test/app-controller-test.js

@@ -53,7 +53,7 @@ describe('AppController', function () {
 
     fakeAnnotationUI = {
       tool: 'comment',
-      clearSelectedAnnotations: sandbox.spy()
+      clearSelectedAnnotations: sandbox.spy(),
     };
 
     fakeAuth = {
@@ -66,16 +66,16 @@ describe('AppController', function () {
       all: sandbox.stub().returns([]),
       discard: sandbox.spy(),
       count: sandbox.stub().returns(0),
-      unsaved: sandbox.stub().returns([])
+      unsaved: sandbox.stub().returns([]),
     };
 
     fakeFeatures = {
       fetch: sandbox.spy(),
-      flagEnabled: sandbox.stub().returns(false)
+      flagEnabled: sandbox.stub().returns(false),
     };
 
     fakeLocation = {
-      search: sandbox.stub().returns({})
+      search: sandbox.stub().returns({}),
     };
 
     fakeParams = {id: 'test'};
@@ -90,7 +90,7 @@ describe('AppController', function () {
 
     fakeWindow = {
       top: {},
-      confirm: sandbox.stub()
+      confirm: sandbox.stub(),
     };
 
     fakeSettings = {

h/static/scripts/test/build-thread-test.js

@@ -81,7 +81,7 @@ describe('build-thread', function () {
         references: ['1'],
       },{
         id: '3',
-        references: ['1','2']
+        references: ['1','2'],
       }];
 
       var thread = createThread(NESTED_FIXTURE);
@@ -92,8 +92,8 @@ describe('build-thread', function () {
           children: [{
             annotation: NESTED_FIXTURE[2],
             children: [],
-          }]
-        }]
+          }],
+        }],
       }]);
     });
 
@@ -210,7 +210,7 @@ describe('build-thread', function () {
         filterFn: function (annot) {
           return annot.text.match(/first/);
         },
-        forceVisible: ['3']
+        forceVisible: ['3'],
       });
       assert.isFalse(thread.children[0].collapsed);
     });
@@ -252,14 +252,14 @@ describe('build-thread', function () {
     context('when there is a selection', function () {
       it('shows only selected annotations', function () {
         var thread = createThread(SIMPLE_FIXTURE, {
-          selected: ['1']
+          selected: ['1'],
         });
         assert.deepEqual(thread, [{
           annotation: SIMPLE_FIXTURE[0],
           children: [{
             annotation: SIMPLE_FIXTURE[2],
             children: [],
-          }]
+          }],
         }]);
       });
     });
@@ -284,7 +284,7 @@ describe('build-thread', function () {
 
         assert.deepEqual(thread, [{
           annotation: fixture[1],
-          children: []
+          children: [],
         }]);
       });
     });
@@ -322,14 +322,14 @@ describe('build-thread', function () {
       },{
         id: '3',
         references: ['1'],
-        created: 100
+        created: 100,
       },{
         id: '2',
         references: ['1'],
         created: 50,
       }];
       var thread = createThread(fixture, {
-        sortCompareFn: function (a, b) { return a.id < b.id; }
+        sortCompareFn: function (a, b) { return a.id < b.id; },
       });
       assert.deepEqual(annots(thread[0].children),
         [fixture[2], fixture[1]]);
@@ -349,7 +349,7 @@ describe('build-thread', function () {
       },{
         annotation: SIMPLE_FIXTURE[1],
         children: [],
-        replyCount: 0
+        replyCount: 0,
       }]);
     });
   });

h/static/scripts/test/groups-test.js

@@ -11,8 +11,8 @@ var sessionWithThreeGroups = function() {
         {name: 'Group 1', id: 'id1'},
         {name: 'Group 2', id: 'id2'},
         {name: 'Group 3', id: 'id3'},
-      ]
-    }
+      ],
+    },
   };
 };
 
@@ -22,7 +22,7 @@ describe('groups', function() {
   var fakeRootScope;
   var fakeHttp;
   var fakeSettings = {
-    serviceUrl: 'https://test.hypothes.is/'
+    serviceUrl: 'https://test.hypothes.is/',
   };
   var sandbox;
 
@@ -32,7 +32,7 @@ describe('groups', function() {
     fakeSession = sessionWithThreeGroups();
     fakeLocalStorage = {
       getItem: sandbox.stub(),
-      setItem: sandbox.stub()
+      setItem: sandbox.stub(),
     };
     fakeRootScope = {
       eventCallbacks: [],
@@ -43,7 +43,7 @@ describe('groups', function() {
         if (event === events.GROUPS_CHANGED) {
           this.eventCallbacks.push(callback);
         }
-      }
+      },
     };
     fakeHttp = sandbox.stub();
   });
@@ -73,7 +73,7 @@ describe('groups', function() {
       assert.deepEqual(groups, [
         {name: 'Group 1', id: 'id1'},
         {name: 'Group 2', id: 'id2'},
-        {name: 'Group 3', id: 'id3'}
+        {name: 'Group 3', id: 'id3'},
       ]);
     });
   });
@@ -177,7 +177,7 @@ describe('groups', function() {
       s.leave('id2');
       assert.calledWithMatch(fakeHttp, {
         url: fakeSettings.serviceUrl + 'groups/id2/leave',
-        method: 'POST'
+        method: 'POST',
       });
     });
   });

h/static/scripts/test/markdown-commands-test.js

@@ -111,7 +111,7 @@ describe('markdown commands', function () {
         'one [' + sel + '](<sel>http://insert-your-link-here.com</sel>) three');
     },[
       {selection: 'two'},
-      {selection: 'jim:smith'}
+      {selection: 'jim:smith'},
     ]);
 
     unroll('converts URLs to links', function (testCase) {

h/static/scripts/test/raven-test.js

@@ -10,9 +10,9 @@ function fakeExceptionData(scriptURL) {
         stacktrace: {
           frames: [{
             filename: scriptURL,
-          }]
-        }
-      }]
+          }],
+        },
+      }],
     },
     culprit: scriptURL,
   };

h/static/scripts/test/root-thread-test.js

@@ -148,7 +148,7 @@ describe('rootThread', function () {
 
     function targetWithPos(pos) {
       return [{
-        selector: [{type: 'TextPositionSelector', start: pos}]
+        selector: [{type: 'TextPositionSelector', start: pos}],
       }];
     }
 

h/static/scripts/test/session-test.js

@@ -63,8 +63,8 @@ describe('h:session', function () {
     it('should invoke the flash service with any flash messages', function () {
       var response = {
         flash: {
-          error: ['fail']
-        }
+          error: ['fail'],
+        },
       };
       $httpBackend.expectPOST(url).respond(response);
       session.login({});
@@ -75,12 +75,12 @@ describe('h:session', function () {
     it('should assign errors and status reasons to the model', function () {
       var response = {
         model: {
-          userid: 'alice'
+          userid: 'alice',
         },
         errors: {
-          password: 'missing'
+          password: 'missing',
         },
-        reason: 'bad credentials'
+        reason: 'bad credentials',
       };
       $httpBackend.expectPOST(url).respond(response);
       var result = session.login({});
@@ -95,7 +95,7 @@ describe('h:session', function () {
       var headers = {
         'Accept': 'application/json, text/plain, */*',
         'Content-Type': 'application/json;charset=utf-8',
-        'X-XSRF-TOKEN': token
+        'X-XSRF-TOKEN': token,
       };
       var model = {csrf: token};
       $httpBackend.expectPOST(url).respond({model: model});
@@ -111,8 +111,8 @@ describe('h:session', function () {
     it('should expose the model as session.state', function () {
       var response = {
         model: {
-          userid: 'alice'
-        }
+          userid: 'alice',
+        },
       };
       assert.deepEqual(session.state, {});
       $httpBackend.expectPOST(url).respond(response);
@@ -185,7 +185,7 @@ describe('h:session', function () {
       $rootScope.$on(events.SESSION_CHANGED, sessionChangeCallback);
       session.update({
         groups: [{
-          id: 'groupid'
+          id: 'groupid',
         }],
         csrf: 'dummytoken',
       });
@@ -196,9 +196,9 @@ describe('h:session', function () {
       sessionChangeCallback.reset();
       session.update({
         groups: [{
-          id: 'groupid2'
+          id: 'groupid2',
         }],
-        csrf: 'dummytoken'
+        csrf: 'dummytoken',
       });
       assert.calledWith(sessionChangeCallback, sinon.match({}),
         {initialLoad: false});
@@ -209,9 +209,9 @@ describe('h:session', function () {
       $rootScope.$on(events.GROUPS_CHANGED, groupChangeCallback);
       session.update({
         groups: [{
-          id: 'groupid'
+          id: 'groupid',
         }],
-        csrf: 'dummytoken'
+        csrf: 'dummytoken',
       });
       assert.calledOnce(groupChangeCallback);
     });
@@ -221,7 +221,7 @@ describe('h:session', function () {
       $rootScope.$on(events.USER_CHANGED, userChangeCallback);
       session.update({
         userid: 'fred',
-        csrf: 'dummytoken'
+        csrf: 'dummytoken',
       });
       assert.calledOnce(userChangeCallback);
     });

h/static/scripts/test/store-test.js

@@ -98,11 +98,11 @@ describe('store', function () {
     var annotation = {
       $highlight: true,
       $notme: 'nooooo!',
-      allowed: 123
+      allowed: 123,
     };
     store.annotation.create({}, annotation);
     $httpBackend.expectPOST('http://example.com/api/annotations', {
-      allowed: 123
+      allowed: 123,
     })
     .respond(function () { return {id: 'test'}; });
     $httpBackend.flush();

h/static/scripts/test/streamer-test.js

@@ -53,7 +53,7 @@ describe('streamer', function () {
     fakeRootScope = {
       $apply: function (callback) {
         callback();
-      }
+      },
     };
 
     fakeAnnotationMapper = {
@@ -110,8 +110,8 @@ describe('streamer', function () {
           action: 'create',
         },
         payload: [{
-          group: 'public'
-        }]
+          group: 'public',
+        }],
       });
       assert.ok(fakeAnnotationMapper.loadAnnotations.calledOnce);
     });
@@ -124,8 +124,8 @@ describe('streamer', function () {
           action: 'delete',
         },
         payload: [{
-          group: 'public'
-        }]
+          group: 'public',
+        }],
       });
       assert.ok(fakeAnnotationMapper.unloadAnnotations.calledOnce);
     });
@@ -136,8 +136,8 @@ describe('streamer', function () {
       createDefaultStreamer();
       var model = {
         groups: [{
-          id: 'new-group'
-        }]
+          id: 'new-group',
+        }],
       };
       fakeWebSocket.notify({
         type: 'session-change',

h/static/scripts/test/time-test.js

@@ -22,7 +22,7 @@ var FIXTURES_TO_FUZZY_STRING = [
   [1 * year, '1 Jan 1970'],
   [1 * year + 2 * month, '1 Jan 1970'],
   [2 * year, '1 Jan 1970'],
-  [8 * year, '1 Jan 1970']
+  [8 * year, '1 Jan 1970'],
 ];
 
 var FIXTURES_NEXT_FUZZY_UPDATE = [
@@ -35,7 +35,7 @@ var FIXTURES_NEXT_FUZZY_UPDATE = [
   [27 * hour, null],
   [3 * day + 30 * minute, null],
   [6 * month + 2 * day, null],
-  [8 * year, null]
+  [8 * year, null],
 ];
 
 describe('time', function () {
@@ -69,9 +69,9 @@ describe('time', function () {
               } else {
                 return '1 Jan 1970';
               }
-            }
+            },
           };
-        }
+        },
       };
     }
 

h/static/scripts/test/widget-controller-test.js

@@ -81,7 +81,7 @@ describe('WidgetController', function () {
 
     fakeAnnotationMapper = {
       loadAnnotations: sandbox.spy(),
-      unloadAnnotations: sandbox.spy()
+      unloadAnnotations: sandbox.spy(),
     };
 
     fakeCrossFrame = {
@@ -98,13 +98,13 @@ describe('WidgetController', function () {
     };
 
     fakeStreamer = {
-      setConfig: sandbox.spy()
+      setConfig: sandbox.spy(),
     };
 
     fakeStreamFilter = {
       resetFilter: sandbox.stub().returnsThis(),
       addClause: sandbox.stub().returnsThis(),
-      getFilter: sandbox.stub().returns({})
+      getFilter: sandbox.stub().returns({}),
     };
 
     fakeGroups = {
@@ -334,7 +334,7 @@ describe('WidgetController', function () {
 
     it('does not clear the selection if the new annotation is a reply', function () {
       $rootScope.$emit('beforeAnnotationCreated', {
-        references: ['parent-id']
+        references: ['parent-id'],
       });
       assert.notCalled($scope.clearSelection);
     });
@@ -353,7 +353,7 @@ describe('WidgetController', function () {
         {
           uri: 'http://www.example.com',
           searchUris: [],
-        }
+        },
       ];
 
       // There is a direct-linked annotation
@@ -393,7 +393,7 @@ describe('WidgetController', function () {
 
     it('shows logged out message if selection is available', function () {
       $scope.auth = {
-        status: 'logged-out'
+        status: 'logged-out',
       };
       annotationUI.addAnnotations([{id: '123'}]);
       annotationUI.selectAnnotations(['123']);
@@ -403,7 +403,7 @@ describe('WidgetController', function () {
 
     it('does not show loggedout message if selection is unavailable', function () {
       $scope.auth = {
-        status: 'logged-out'
+        status: 'logged-out',
       };
       annotationUI.selectAnnotations(['missing']);
       $scope.$digest();
@@ -412,7 +412,7 @@ describe('WidgetController', function () {
 
     it('does not show loggedout message if there is no selection', function () {
       $scope.auth = {
-        status: 'logged-out'
+        status: 'logged-out',
       };
       annotationUI.selectAnnotations([]);
       $scope.$digest();
@@ -421,7 +421,7 @@ describe('WidgetController', function () {
 
     it('does not show loggedout message if user is not logged out', function () {
       $scope.auth = {
-        status: 'logged-in'
+        status: 'logged-in',
       };
       annotationUI.addAnnotations([{id: '123'}]);
       annotationUI.selectAnnotations(['123']);
@@ -431,7 +431,7 @@ describe('WidgetController', function () {
 
     it('does not show loggedout message if not a direct link', function () {
       $scope.auth = {
-        status: 'logged-out'
+        status: 'logged-out',
       };
       delete fakeSettings.annotations;
       annotationUI.addAnnotations([{id: '123'}]);

h/static/scripts/time.js

@@ -103,33 +103,33 @@ var BREAKPOINTS = [
   {
     test: lessThanThirtySecondsAgo,
     format: function () {return 'Just now';},
-    nextUpdate: 1
+    nextUpdate: 1,
   },
   {
     test: lessThanOneMinuteAgo,
     format: nSec,
-    nextUpdate: 1
+    nextUpdate: 1,
   },
   {
     test: lessThanOneHourAgo,
     format: nMin,
-    nextUpdate: minute
+    nextUpdate: minute,
   },
   {
     test: lessThanOneDayAgo,
     format: nHr,
-    nextUpdate: hour
+    nextUpdate: hour,
   },
   {
     test: thisYear,
     format: dayAndMonth,
-    nextUpdate: null
+    nextUpdate: null,
   },
   {
     test: function () {return true;},
     format: dayAndMonthAndYear,
-    nextUpdate: null
-  }
+    nextUpdate: null,
+  },
 ];
 
 function getBreakpoint(date, now) {

h/static/scripts/util/observable.js

@@ -76,7 +76,7 @@ function delay(delay, src) {
          lastValue = value;
          clearTimeout(timeout);
          timeout = setTimeout(onNext, delay);
-       }
+       },
      });
 
      return function () {

h/static/scripts/websocket.js

@@ -50,7 +50,7 @@ function Socket(url) {
       // Don't retry forever -- fail permanently after 10 retries
       retries: 10,
       // Randomize retry times to minimise the thundering herd effect
-      randomize: true
+      randomize: true,
     });
 
     operation.attempt(function () {

scripts/decaf/decaf.js

@@ -53,7 +53,7 @@ var stripReturnPatterns = [
   // Unit test cases
   /it\(/,
   // Assignments in setters etc.
-  /[^=]+=[^=]+/
+  /[^=]+=[^=]+/,
 ];
 
 /**

scripts/gulp/upload-to-sentry.js

@@ -73,7 +73,7 @@ function uploadReleaseFile(opts, project, release, file) {
     formData: {
       file: fs.createReadStream(file.path),
       name: path.basename(file.path),
-    }
+    },
   }).then(function (result) {
     if (result.response.statusCode === 201) {
       return;

scripts/gulp/vendor-bundles.js

@@ -19,7 +19,7 @@ module.exports = {
       'ng-tags-input',
       'angular-toastr',
       'angulartics/src/angulartics',
-      'angulartics/src/angulartics-ga'
+      'angulartics/src/angulartics-ga',
     ],
     katex: ['katex'],
     showdown: ['showdown'],
@@ -39,5 +39,5 @@ module.exports = {
   noParseModules: [
     'jquery',
     'katex',
-  ]
+  ],
 };