Skip to content

fix(compat): update patch for typescript@5.7.1-rc#6591

Merged
arcanis merged 2 commits intomasterfrom
merceyz/fix/typescript-57-rc
Nov 21, 2024
Merged

fix(compat): update patch for typescript@5.7.1-rc#6591
arcanis merged 2 commits intomasterfrom
merceyz/fix/typescript-57-rc

Conversation

@merceyz
Copy link
Copy Markdown
Member

@merceyz merceyz commented Nov 9, 2024

What's the problem this PR addresses?

The PnP compatibility patch for TypeScript doesn't apply to typescript@5.7.1-rc.

Ref microsoft/TypeScript#35206

How did you fix it?

Rebased it.

Checklist

  • I have read the Contributing Guide.
  • 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.

@arcanis arcanis merged commit aa557fe into master Nov 21, 2024
@arcanis arcanis deleted the merceyz/fix/typescript-57-rc branch November 21, 2024 22:17
merceyz added a commit that referenced this pull request Dec 4, 2024
**What's the problem this PR addresses?**

The PnP compatibility patch for TypeScript doesn't apply to
`typescript@5.7.1-rc`.

Ref microsoft/TypeScript#35206

**How did you fix it?**

Rebased it.

**Checklist**
- [x] I have read the [Contributing
Guide](https://yarnpkg.com/advanced/contributing).
- [x] I have set the packages that need to be released for my changes to
be effective.
- [x] I will check that all automated PR checks pass before the PR gets
reviewed.
@devinrhode2
Copy link
Copy Markdown
Contributor

devinrhode2 commented Jul 11, 2025

I wish I could use yarn v4 without requiring usage of this typescript patch. A new team I joined is using yarn v1 on a new nx monorepo and it would be nice if we could use a faster version of yarn; but I regard the typescript patch as a non-started based on my yarn v2/v3 experiences.

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