eslint - remove import/no-extraneous-dependencies & add ignore

- ignoring in one location where import is added for dev only

.eslintrc.js

@@ -5,7 +5,6 @@ const legacyCode = {
   'constructor-super': 'off',
   'default-param-last': 'off',
   'import/no-cycle': 'off',
-  'import/no-extraneous-dependencies': 'off',
   'jsx-a11y/alt-text': 'off',
   'jsx-a11y/anchor-is-valid': 'off',
   'jsx-a11y/click-events-have-key-events': 'off',

client/src/entrypoints/admin/wagtailadmin.js

@@ -9,7 +9,7 @@ import initCollapsibleBreadcrumbs from '../../includes/breadcrumbs';
 if (process.env.NODE_ENV === 'development') {
   // Run react-axe in development only, so it does not affect performance
   // in production, and does not break unit tests either.
-  // eslint-disable-next-line global-require, @typescript-eslint/no-var-requires
+  // eslint-disable-next-line global-require, @typescript-eslint/no-var-requires, import/no-extraneous-dependencies
   const axe = require('react-axe');
   axe(React, ReactDOM, 1000);
 }