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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add dry-run #3300

Merged
merged 1 commit into from
May 25, 2023
Merged

chore: add dry-run #3300

merged 1 commit into from
May 25, 2023

Conversation

hardfist
Copy link
Contributor

@hardfist hardfist commented May 25, 2023

Related issue (if exists)

Summary

馃 Generated by Copilot at 33d6db0

This pull request improves the code quality and the publish process of the x.mjs file. It formats the code with prettier, adds a --dry-run option to the publish command, and switches to pnpm as the package manager.

Walkthrough

馃 Generated by Copilot at 33d6db0

  • Add a new script to run prettier on x.mjs (link)
  • Switch to pnpm for publishing packages and add a --dry-run option (link, link)
  • Format x.mjs with prettier and pass --dry-run option to publish command (link, link)

@hardfist hardfist requested review from Boshen and hyf0 as code owners May 25, 2023 09:24
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label May 25, 2023
@hardfist hardfist added this pull request to the merge queue May 25, 2023
Merged via the queue into main with commit 642029f May 25, 2023
16 checks passed
@hardfist hardfist deleted the add-dry-run branch May 25, 2023 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants