-
-
Notifications
You must be signed in to change notification settings - Fork 609
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
feat: output pnpm version with info
/version
command
#3906
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3906 +/- ##
=======================================
Coverage 90.87% 90.87%
=======================================
Files 22 22
Lines 1688 1688
Branches 486 486
=======================================
Hits 1534 1534
Misses 154 154
Continue to review full report in Codecov by Sentry.
|
Is this still a draft? |
Yeah. I just need to fix the tests on CI, will do it today. |
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.
lgtm
Could you rebase to make commitlint happy? |
@evenstensberg I'll modify the commit message while merging the PR, it should be fine. |
What kind of change does this PR introduce?
feature
Did you add tests for your changes?
Yes
If relevant, did you update the documentation?
No
Summary
Fixes #3072
Add
pnpm
informationDoes this PR introduce a breaking change?
No
Other information
No