-
Notifications
You must be signed in to change notification settings - Fork 139
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
docs: Replace pnpx
with pnpm dlx
#527
Conversation
✅ Deploy Preview for creative-fairy-df92c4 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
pnpx
with pnpm dlx
pnpx
with pnpm dlx
@lucadalli Can you replace the other instances of |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #527 +/- ##
==========================================
- Coverage 86.84% 86.55% -0.30%
==========================================
Files 111 111
Lines 8863 8864 +1
Branches 879 876 -3
==========================================
- Hits 7697 7672 -25
- Misses 1152 1178 +26
Partials 14 14 ☔ View full report in Codecov by Sentry. |
If you allow maintainers to contribute to your PR, I can add the rest of the changes. |
@aklinker1 Sorry for the incomplete PR, I changed the rest. Edit: Just an fyi, after switching to node 18.16.1 the original command |
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.
Nope, I'll keep this change. Thanks for updating the other references!
pnpx
with pnpm dlx
pnpx
with pnpm dlx
After this modification, maybe the homepage video can also be updated? @aklinker1 |
Faced with the error below when trying to use
pnpx
on my Windows machine.pnpx : The term 'pnpx' is not recognized as the name of a cmdlet, function, script file, or operable program.
pnpm dlx
works fine. pnpm maker Zoltan sayspnpx
is being deprecated in favor ofpnpm dlx
.