Auto merge of #2046 - jtgeibel:eslint-overrides, r=Turbo87

Remove some unnecessary eslint overrides

I was experimenting with the eslint setup, and it appears that some lint overrides are no longer needed.

This PR removes all results for `git grep "eslint-disable "` and removes some overrides in `.template-lintrc.js`.  There are 10 remaining needed `eslint-disable-next-line` usages.

r? @Turbo87

Author: bors

.template-lintrc.js

@@ -9,9 +9,6 @@ module.exports = {
     'require-valid-alt-text': false,
     'no-action': false,
     'no-curly-component-invocation': false,
-    'no-html-comments': false,
     'no-implicit-this': false,
-    'no-unnecessary-concat': false,
-    quotes: false,
   },
 };

fastboot.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 /* eslint-env node */
 
 'use strict';

mirage/fixtures/categories.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     category: 'API bindings',

mirage/fixtures/crates.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     badges: [

mirage/fixtures/dependencies.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     crate_id: 'libc',

mirage/fixtures/keywords.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     crates_cnt: 38,

mirage/fixtures/teams.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     avatar: 'https://avatars.githubusercontent.com/u/565790?v=3',

mirage/fixtures/users.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     avatar: 'https://avatars0.githubusercontent.com/u/9447137?v=3',

mirage/fixtures/version-downloads.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     date: '2017-02-10T00:00:00Z',

mirage/fixtures/versions.js

@@ -1,4 +1,3 @@
-/* eslint-disable quotes */
 export default [
   {
     crate: 'nanomsg',