cerner / terra-toolkit

Terra Toolkit is a monorepo that contains utility modules for use when developing using Terra components
https://engineering.cerner.com/terra-toolkit
Apache License 2.0
32 stars 37 forks source link

[eslint-config-terra] Update dependencies #822

Closed sdadn closed 10 months ago

sdadn commented 10 months ago

Summary

This PR moves the eslint to peerDependencies. It was accidentally moved to dependencies as part of #811 . This unintended change resulted in webpack failures with NPM 6 as the dependencies weren't getting installed correctly. NPM 9 was not affected due to how it handles dependencies differently.

Testing

This change was tested using:

This change was tested by setting the dev environment to NPM 6 and running npm run clean:install && npm start on terra-framework locally. This resulted in the following failure:

This was again tested by running npm pack and using the local eslint-config-terra with npm install -D path/to/tgz which resulted in the same error.

The error was fixed by moving eslint to peerDepndencies and then repeating the steps above to test the change locally. The result was npm start successfully executing:

Reviews

In addition to engineering reviews, this PR needs:


Thank you for contributing to Terra. @cerner/terra

kenk2 commented 10 months ago

Verified these changes locally: