-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(valid-package-def): ignore pnpm workspace protocol #258
fix(valid-package-def): ignore pnpm workspace protocol #258
Conversation
pnpm has a "workspace protocol" (link) that allows you to explicitly say a package lives in your current workspace. Ignore this when doing package.json validation. This fixes JoshuaKGoldberg#251
f210640
to
d4adc8f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 thanks!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #258 +/- ##
==========================================
- Coverage 99.30% 99.27% -0.04%
==========================================
Files 15 17 +2
Lines 866 965 +99
Branches 73 81 +8
==========================================
+ Hits 860 958 +98
- Misses 6 7 +1 ☔ View full report in Codecov by Sentry. |
@all-contributors please add @altano for code.
|
I've put up a pull request to add @altano! 🎉 I couldn't determine any contributions to add, did you specify any contributions? |
🎉 This is included in version v0.12.1 🎉 The release is available on: Cheers! 📦🚀 |
Adds @altano as a contributor for code. This was requested by JoshuaKGoldberg [in this comment](#258 (comment)) --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
PR Checklist
status: accepting prs
Overview
pnpm has a "workspace protocol" (link) that allows you to explicitly say a package lives in your current workspace. Ignore this when doing package.json validation by adding
workspace
to the ignore glob.