Skip to content

chore: shorten pnpm dlx to pnpx#670

Merged
harlan-zw merged 1 commit intomainfrom
chore/pnpx-alias
Mar 24, 2026
Merged

chore: shorten pnpm dlx to pnpx#670
harlan-zw merged 1 commit intomainfrom
chore/pnpx-alias

Conversation

@DamianGlowala
Copy link
Member

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

@vercel
Copy link
Contributor

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
scripts-playground Ready Ready Preview, Comment Mar 24, 2026 9:19am

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 24, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/scripts@670

commit: e0d9e2f

@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 832002c4-c45a-4541-ba28-012895fd18a5

📥 Commits

Reviewing files that changed from the base of the PR and between 58981fe and e0d9e2f.

📒 Files selected for processing (1)
  • .github/workflows/nightly.yml

📝 Walkthrough

Walkthrough

The pull request modifies the .github/workflows/nightly.yml file, specifically updating the "Publish" step of the nightly workflow. The change replaces the package runner invocation from pnpm dlx pkg-pr-new publish --pnpm to pnpx pkg-pr-new publish --pnpm. This affects how the pkg-pr-new package is executed during the nightly build pipeline, with all other command arguments remaining unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description follows the template structure and correctly identifies the change as a chore, but the Description section is empty with no explanation of why this change was made or what problem it solves. Fill in the Description section to explain the rationale for this change and any benefits (e.g., improving readability or following conventions).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: shorten pnpm dlx to pnpx' directly and accurately describes the main change: replacing the longer pnpm command invocation with its shorter alias.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pnpx-alias

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@harlan-zw harlan-zw merged commit 1bcf4bc into main Mar 24, 2026
10 checks passed
@harlan-zw harlan-zw deleted the chore/pnpx-alias branch March 24, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants