fix: Ignoring prod subdependencies without ropm keyword

rokucommunity / ropm

A package manager for the Roku platform.

MIT License

31 stars 5 forks source link

fix: Ignoring prod subdependencies without ropm keyword #44

Closed pawelhertman closed 2 years ago

pawelhertman commented 3 years ago

Currently, if a prod dependency has another prod dependency that has no "ropm" keyword, ROPM is crashing. Since NPM v7, the default depth is 1. This change doesn't break on NPM v6 (which has infinity depth by default)

pawelhertman commented 3 years ago

@TwitchBronBron I updated PR's description

pawelhertman commented 2 years ago

@TwitchBronBron any chance to review my 3 PRs soon?

TwitchBronBron commented 2 years ago

I had hoped to get to them this weekend, but the whole family is sick right now. I'll try real hard to get to them in the next week or so.