Skip to content

remove yarn from default-trusted-dependencies.txt #20684

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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 3, 2025

Conversation

RiskyMH
Copy link
Member

@RiskyMH RiskyMH commented Jun 27, 2025

What does this PR do?

Yarn has a postinstall script but its not needed when using bun. The script is about corepack stuff and ensuring its not conflicting, however the way it is checking means it should never be affected by bun i -g yarn or bunx yarn. As such bun is just slowing down trying to check for it.

https://app.unpkg.com/yarn@1.22.22/files/preinstall.js

As a sideresult this should make it work on windows, even though the actual fix for it is to make bun shell understand :; (node ./preinstall.js > /dev/null 2>&1 || true)

bunx yarn@latest

  ⚙️  yarn [1/1] error: Failed to run script C:\Users\...\AppData\Local\Temp\bunx-3645674270-yarn@latest\node_modules\yarn\[eval] due to error expected a command or assignment but got: "Redirect"
  • Documentation or TypeScript types (it's okay to leave the rest blank in this case)
  • Code changes

How did you verify your code works?

@robobun
Copy link
Collaborator

robobun commented Jun 27, 2025

Updated 4:04 AM PT - Jun 27th, 2025

@RiskyMH, your commit 28da074 has 1 failures in Build #19375:


🧪   To try this PR locally:

bunx bun-pr 20684

That installs a local version of the PR into your bun-20684 executable, so you can run:

bun-20684 --bun

@nektro nektro merged commit 8657d70 into main Jul 3, 2025
60 of 62 checks passed
@nektro nektro deleted the RiskyMH/rm-yarn-trusted-deps branch July 3, 2025 23:23
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