unjs / nypm

๐ŸŒˆ Unified Package Manager for Node.js and Bun
MIT License
471 stars 17 forks source link

fix: use correct flag when getting pnpm workspace args #130

Closed DamianGlowala closed 3 months ago

DamianGlowala commented 4 months ago

๐Ÿ”— Linked issue

resolves https://github.com/unjs/nypm/issues/123

โ“ Type of change

๐Ÿ“š Description

Uses correct flag when getting pnpm workspace args (-F/--filter).

๐Ÿ“ Checklist

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.66%. Comparing base (660392f) to head (d9a3b23). Report is 23 commits behind head on main.

:exclamation: Current head d9a3b23 differs from pull request most recent head 7ac18e0

Please upload reports for the commit 7ac18e0 to get more accurate results.

:exclamation: There is a different number of reports uploaded between BASE (660392f) and HEAD (d9a3b23). Click for more details.

HEAD has 1 upload less than BASE | Flag | BASE (660392f) | HEAD (d9a3b23) | |------|------|------| ||3|2|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #130 +/- ## ========================================== - Coverage 82.17% 72.66% -9.51% ========================================== Files 6 6 Lines 516 633 +117 Branches 71 83 +12 ========================================== + Hits 424 460 +36 - Misses 91 170 +79 - Partials 1 3 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

pi0 commented 3 months ago

Thanks!