Skip to content

Conversation

@paul-soporan
Copy link
Member

What's the problem this PR addresses?

We still remove the old .pnp.js files when migrating, but our telemetry currently shows that only 5% of all modern users still use Yarn 2, so it doesn't make sense for us to still keep this migration feature one more major when they can just migrate one major at a time.

How did you fix it?

Removed the code.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@paul-soporan paul-soporan mentioned this pull request Aug 17, 2022
13 tasks
@arcanis arcanis merged commit d972863 into master Aug 18, 2022
@arcanis arcanis deleted the paul/refactor/dont-remove-pnpjs branch August 18, 2022 09:28
@Tommybio

This comment was marked as abuse.

@merceyz merceyz added this to the 4.0.0 milestone May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants