Skip to content

chore: Fix Renovate PR builds#269

Merged
liamhughes merged 1 commit intomainfrom
liamhughes/fix-renovate-pr-artifact-updates
Apr 23, 2026
Merged

chore: Fix Renovate PR builds#269
liamhughes merged 1 commit intomainfrom
liamhughes/fix-renovate-pr-artifact-updates

Conversation

@liamhughes
Copy link
Copy Markdown
Contributor

@liamhughes liamhughes commented Apr 23, 2026

Our Renovate PRs are displaying the following error:

Command failed: npm i -g pnpm@8.15.9 && pnpm install && pnpm build                                                                                                                                                                                                           
npm error code EINVALIDTAGNAME                                                                                                                                                                                                                                               
npm error Invalid tag name "&&" of package "&&": Tags may not have any characters that encodeURIComponent encodes.                                                                                                                                                           
npm error A complete log of this run can be found in: /home/ubuntu/.npm/_logs/2026-03-17T00_18_08_462Z-debug-0.log  

Example: #267 (comment)

The postUpgradeTasks.commands field takes an array of commands.

@liamhughes liamhughes requested a review from a team as a code owner April 23, 2026 06:31
@liamhughes liamhughes requested review from Copilot and removed request for a team April 23, 2026 06:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes Renovate PR build failures by aligning postUpgradeTasks.commands with Renovate’s expected format (an array of individual commands rather than a single shell-chained command).

Changes:

  • Split the post-upgrade task from one &&-chained command into three discrete commands (npm i -g ..., pnpm install, pnpm build).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@liamhughes liamhughes merged commit 97579f6 into main Apr 23, 2026
12 checks passed
@liamhughes liamhughes deleted the liamhughes/fix-renovate-pr-artifact-updates branch April 24, 2026 01:25
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.

3 participants