-
Notifications
You must be signed in to change notification settings - Fork 29
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(command): support --lib arguments #390
Conversation
✅ Deploy Preview for rslib ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
CodSpeed Performance ReportMerging #390 will not alter performanceComparing Summary
|
@@ -58,6 +72,11 @@ export function runCli(): void { | |||
|
|||
inspectCommand | |||
.description('inspect the Rsbuild / Rspack configs of Rslib projects') | |||
.option( | |||
'--lib <name>', |
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.
shared options can be set in CommonOptions
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.
There's a bit difference in the text.
'build the specified library (may be repeated)',
'inspect the specified library (may be repeated)',
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.
use choose the specified library
is enough, and what is may be repeated
mean?
@@ -58,6 +59,7 @@ inspect the Rsbuild / Rspack configs of Rslib projects | |||
Options: | |||
-c --config <config> specify the configuration file, can be a relative or absolute path | |||
--env-mode <mode> specify the env mode to load the `.env.[mode]` file | |||
--lib <name> build the specified library (may be repeated) |
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.
We should add docs about what name is.
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.
In following PR.
Summary
Close #382.
Support following CLI args for
build
andinspect
, only build specified lib. Could also be used withrslib mf dev
in #349.Related Links
Checklist