Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

please move snyk from dependencies to devDependencies

See original GitHub issue

Is snyk required to use quagga2? If not, it’ll be amazing if it wasn’t part of npm install @ericblade/quagga2. I ask because snyk is a pretty large dependency, weighting in at minimum 50 MB, up to 120 MB extra - depending how many of the peers of snyk you already have.

Issue Analytics

State:
Created 2 years ago
Reactions:1
Comments:13 (7 by maintainers)

Top GitHub Comments

2reactions

ericbladecommented, Jul 11, 2021

just went ahead and removed it in https://github.com/ericblade/quagga2/commit/c966812ee968ad8a00d85b66d59741149adf0270

also updated all the other safe to update devDeps

2reactions

ericbladecommented, Jul 2, 2021

I appreciate the additional analysis. The situation has quite probably changed quite a bit from when I forked initially.

Good news? i’ve got a long weekend from work right now, perhaps between time spent catching up on house repairs and mandatory computer upgrades, I’ll grab some time to take a look at this. It might well be possible that just relying on dependabot would be fine… the landscape for this sort of stuff has changed drastically in the last couple of years. Maybe i’ll get the automated release/patch notes things working correctly too 😃