Skip to content

Use pinned version of pkg-pr-new for preview releases - #4111

Merged
HiDeoo merged 1 commit into
withastro:mainfrom
HiDeoo:hd/fix-pkg-pr-new-dlx
Aug 4, 2026
Merged

HiDeoo merged 1 commit into
withastro:mainfrom
HiDeoo:hd/fix-pkg-pr-new-dlx

Conversation

@HiDeoo

@HiDeoo HiDeoo commented Aug 4, 2026 •

Copy link
Copy Markdown
Member

Description

While reviewing another PR, I noticed we use pnpm dlx to run pkg-pr-new for preview releases.

This is not recommended because pnpm dlx would always resolves to the latest version which could have unexpected behavior due to some upstream breaking changes but also increases supply-chain risks.

Caution

In CI environments, avoid npx, pnpm dlx, yarn dlx, and bunx for this step. Install pkg-pr-new as a dependency and execute it from the lockfile (npm exec, pnpm exec, yarn, or bun run).

This PR fixes that by installing pkg-pr-new as a dev dependency and running it using pnpm exec.

@changeset-bot

changeset-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 389fed9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify

netlify Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for astro-starlight ready!

Name Link
🔨 Latest commit 389fed9
🔍 Latest deploy log https://app.netlify.com/projects/astro-starlight/deploys/6a71b4161e7e7a00085983de
😎 Deploy Preview https://deploy-preview-4111--astro-starlight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@github-actions github-actions Bot added the 🚨 action Changes to GitHub Action workflows label Aug 4, 2026

@delucis delucis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @HiDeoo!

@HiDeoo
HiDeoo merged commit d393789 into withastro:main Aug 4, 2026
17 checks passed
HiDeoo added a commit to HiDeoo/starlight that referenced this pull request Aug 16, 2026
* main: (219 commits)
  i18n(fr): update `authoring-content.mdx`, `i18n.mdx` & `plugins.md` (withastro#4128)
  i18n(de): update links to Astro docs (withastro#4126)
  Update `size-limit` config (withastro#4122)
  Replace 3 outdated links to Astro Docs (withastro#4120)
  [ci] release (withastro#4115)
  Update dependencies (withastro#4113)
  Add Astro LilyPond to Starlight showcase (withastro#4116)
  Refactor Sätteri RTL code support plugin (withastro#4114)
  Use pinned version of `pkg-pr-new` for preview releases (withastro#4111)
  chore(deps): update vite and rolldown (withastro#4108)
  i18n(ru): update translations (withastro#4107)
  [ci] release (withastro#4106)
  Add Forgejo icon (withastro#4102)
  [ci] format
  i18n(de): update `themes.mdx` (withastro#4104)
  i18n(de): update `plugins.mdx` (withastro#4103)
  i18n(de): update `community-content.mdx` (withastro#4101)
  [ci] release (withastro#4097)
  Fix union schema `extend` regression (withastro#4095)
  [ci] release (withastro#4090)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 action Changes to GitHub Action workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants