eslint-formatter-summary icon indicating copy to clipboard operation
eslint-formatter-summary copied to clipboard

Published 20 hours ago • verified publisher icon mhipszki

Move `np` to devDependencies

Open tlusk opened this issue 3 years ago • 2 comments

The np dependency is only used in the release scripts and should not be marked as an optional runtime dependency.

Having np be installed as an optional runtime dependency is causing warnings on npm and yarn installation of the eslint-formatter-summary package due to issues in the dependencies of np such as security vulnerabilities and warnings about the engines block defined by np.

This resolves #36

tlusk avatar Jan 28 '22 17:01 tlusk

Coverage Status

Coverage remained the same at 100.0% when pulling 55ad451475085ee3d94ab49ff5b6ebfcf6fb51f8 on tlusk:feature-np-devdependency into 2f7bc3cc3f00753f0d2e8d686695811b3af79ef5 on mhipszki:master.

coveralls avatar Jan 28 '22 17:01 coveralls

Hi, could we get this merged? It fixes https://github.com/mhipszki/eslint-formatter-summary/issues/38

MikelArnaiz avatar Feb 03 '23 11:02 MikelArnaiz