Skip to content

fix(zip): List all private packages correctly in a PNPM workspace#520

Merged
aklinker1 merged 1 commit intomainfrom
pnpm-private-packages-in-workspace
Mar 3, 2024
Merged

fix(zip): List all private packages correctly in a PNPM workspace#520
aklinker1 merged 1 commit intomainfrom
pnpm-private-packages-in-workspace

Conversation

@aklinker1
Copy link
Member

@aklinker1 aklinker1 commented Mar 3, 2024

If a private package is in the root package.json, it was not discovered. Now it is, using pnpm ls -r instead of just pnpm ls. This is valid for non-workspace repos as well.

@netlify
Copy link

netlify bot commented Mar 3, 2024

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit d9bcee1
🔍 Latest deploy log https://app.netlify.com/sites/creative-fairy-df92c4/deploys/65e49d3351fa6e0008bfd2a6
😎 Deploy Preview https://deploy-preview-520--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aklinker1 aklinker1 changed the title fix: List all private packages correctly in a pnpm workspace fix: List all private packages correctly in a PNPM workspace Mar 3, 2024
@codecov
Copy link

codecov bot commented Mar 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.48%. Comparing base (f19e690) to head (d9bcee1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
- Coverage   86.55%   86.48%   -0.07%     
==========================================
  Files         111      111              
  Lines        8860     8860              
  Branches      875      875              
==========================================
- Hits         7669     7663       -6     
- Misses       1177     1183       +6     
  Partials       14       14              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aklinker1 aklinker1 changed the title fix: List all private packages correctly in a PNPM workspace fix(zip): List all private packages correctly in a PNPM workspace Mar 3, 2024
@aklinker1 aklinker1 marked this pull request as ready for review March 3, 2024 15:56
@aklinker1 aklinker1 merged commit aabba41 into main Mar 3, 2024
@aklinker1 aklinker1 deleted the pnpm-private-packages-in-workspace branch March 3, 2024 15:56
@aklinker1
Copy link
Member Author

Released in v0.17.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant