eslint-plugin-unicorn: Disable new rules

Author: Turbo87

.eslintrc.js

@@ -56,6 +56,8 @@ module.exports = {
     'unicorn/no-nested-ternary': 'off',
     // disabled because of unfixable false positives
     'unicorn/prevent-abbreviations': 'off',
+    // disabled because we are targeting only browsers at the moment
+    'unicorn/prefer-global-this': 'off',
     // disabled because we don't want to go all-in on ES6 modules for Node.js code yet
     'unicorn/prefer-module': 'off',
     // disabled because it seems unnecessary

app/initializers/hashchange.js

@@ -12,6 +12,7 @@ function findElementByFragmentName(document, name) {
   }
 
   try {
+    // eslint-disable-next-line unicorn/prefer-query-selector
     return document.querySelector(`#${name}`) || document.getElementsByName(name)[0];
   } catch {
     // Catches exceptions thrown when an anchor in a readme was invalid (see issue #3108)