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
CLI: Seclude version output functionality out of CLI class #8741
Conversation
7f2e5af
to
cc67f67
Compare
Codecov Report
@@ Coverage Diff @@
## master #8741 +/- ##
=======================================
Coverage 87.43% 87.44%
=======================================
Files 256 257 +1
Lines 9631 9630 -1
=======================================
Hits 8421 8421
+ Misses 1210 1209 -1
Continue to review full report at Codecov.
|
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.
One note about node version check, otherwise it looks good 👍
lib/cli/eventually-list-version.js
Outdated
|
||
const userNodeVersion = Number(process.version.split('.')[0].slice(1)); | ||
// only show components version if user is running Node 8+ | ||
if (userNodeVersion >= 8) { |
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.
I believe this check is redundant as we require node>=10
now
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.
Good find, thanks for pointing!
cc67f67
to
6a091e3
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.
👍
Addresses 1.0 from #8364