Update ESLint rules for scripts (#6081)

Our ESLint rules have been updated to include override rules for
scripts. Scripts run in a Node.js environment where using `console.log`
is expected, so they rules against using Node.js imports and
 `console.log` did not make sense.

This allows us to remove various ESLint ignore rules that were added to
scripts.

.eslintrc.js

@@ -14,6 +14,14 @@ module.exports = {
       files: ['*.{ts,tsx}'],
       extends: ['@metamask/eslint-config-typescript'],
     },
+    {
+      files: ['scripts/**/*.js'],
+      rules: {
+        'no-console': 0,
+        'import/no-commonjs': 0,
+        'import/no-nodejs-modules': 0,
+      },
+    },
   ],
 
   globals: {

scripts/metamask-bot-build-announce-bitrise.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-// eslint-disable-next-line import/no-commonjs
+
 const axios = require('axios');
 
 const ANDROID_APK_LINK = process.env.BITRISE_PUBLIC_INSTALL_PAGE_URL;

scripts/metamask-bot-build-generate-static-assets.js

@@ -1,7 +1,5 @@
 #!/usr/bin/env node
-/* eslint-disable import/no-commonjs */
-/* eslint-disable import/no-nodejs-modules */
-/* eslint-disable no-console */
+
 const contractMetadata = require('@metamask/contract-metadata');
 const fs = require('fs');
 const path = require('path');