Skip to content

Disable `import/no-extraneous-dependencies` rule more narrowly

Mike Greiling requested to merge mg-resolve-eslint-exceptions into master

Right now we turn off import/no-extraneous-dependencies globally in eslintrc.yml but this is not a great practice. The only place where we should allow imports of devDependencies is in modules which are not required to compile and produce this npm package.

I've re-enabled this rule, with a narrow exception for svgpreviewer/* so that it will catch these exceptions correctly. I've also moved some devDependencies over to dependencies where they belong. Finally, I have also resolved two missing dependencies which should prevent yarn from producing warnings on install.

Merge request reports