Skip to content

fix: migrate build-script allow-list to pnpm 11 allowBuilds - #40

Merged
Ahmedhossamdev merged 3 commits into
OneBusAway:mainfrom
0xaboomar:fix/pnpm-11-allowbuilds
Aug 18, 2026
Merged

Ahmedhossamdev merged 3 commits into
OneBusAway:mainfrom
0xaboomar:fix/pnpm-11-allowbuilds

Conversation

@0xaboomar

Copy link
Copy Markdown
Member

Why this PR

The current pnpm.onlyBuiltDependencies configuration is outdated. The official pnpm 10.26 release notes introduce allowBuilds as the preferred replacement for onlyBuiltDependencies and ignoredBuiltDependencies.

Updating this prevents ERR_PNPM_IGNORED_BUILDS for future contributors using newer pnpm versions and keeps the project aligned with the current pnpm configuration.

Changes

  • Replace onlyBuiltDependencies with allowBuilds in pnpm-workspace.yaml
  • Remove the deprecated pnpm.onlyBuiltDependencies configuration
  • Add Node.js >=20.19.0 required by Vite 7

@coderabbitai

coderabbitai Bot commented Aug 16, 2026 •

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 64a06690-2918-4393-ae65-61986bd79ea2


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.

@Ahmedhossamdev

Copy link
Copy Markdown
Member

Nice <3

@Ahmedhossamdev
Ahmedhossamdev merged commit 8a35a48 into OneBusAway:main Aug 18, 2026
3 checks passed
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