Skip to content
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

Merged
merged 1 commit into from
Nov 11, 2024
Merged

feat(command): support --lib arguments #390

merged 1 commit into from
Nov 11, 2024

Conversation

fi3ework
Copy link
Member

@fi3ework fi3ework commented Nov 8, 2024

Summary

Close #382.

Support following CLI args for build and inspect, only build specified lib. Could also be used with rslib mf dev in #349.

rslib build --lib esm
rslib build --lib esm --lib cjs
rslib inspect --lib esm
rslib inspect --lib esm --lib cjs

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link

netlify bot commented Nov 8, 2024

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit 40d9125
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/672e47d503e1310008197581
😎 Deploy Preview https://deploy-preview-390--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codspeed-hq bot commented Nov 8, 2024

CodSpeed Performance Report

Merging #390 will not alter performance

Comparing --lib (40d9125) with main (8bee065)

Summary

✅ 5 untouched benchmarks

@@ -58,6 +72,11 @@ export function runCli(): void {

inspectCommand
.description('inspect the Rsbuild / Rspack configs of Rslib projects')
.option(
'--lib <name>',
Copy link
Collaborator

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

Copy link
Member Author

@fi3ework fi3ework Nov 11, 2024

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)',

Copy link
Collaborator

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)
Copy link
Collaborator

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.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In following PR.

@Timeless0911 Timeless0911 merged commit 30b0ec1 into main Nov 11, 2024
21 checks passed
@Timeless0911 Timeless0911 deleted the --lib branch November 11, 2024 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: named config and positive filter
2 participants