Skip to content

[*] Chore: Update flow-bin to 0.306.0 and add incompatible-variance workaround#8253

Merged
etrepum merged 2 commits intofacebook:mainfrom
etrepum:update-flow-306
Mar 23, 2026
Merged

[*] Chore: Update flow-bin to 0.306.0 and add incompatible-variance workaround#8253
etrepum merged 2 commits intofacebook:mainfrom
etrepum:update-flow-306

Conversation

@etrepum
Copy link
Copy Markdown
Collaborator

@etrepum etrepum commented Mar 23, 2026

Description

Updates flow-bin and configures .flowconfig for 0.306.0 and adds a workaround for the incompatible-variance violation in the signatures for afterCloneFrom and resetOnCopyNodeFrom. TypeScript allows the unsound usage of this as a parameter, which matches the intent of the argument in these functions, but Flow does not… so we do something even more unsound as a workaround.

Note that this same unsound variance usage happens in updateDOM, so we use the same workaround here. If a better workaround is found, all three signatures should be addressed.

Closes #8252

Test plan

pnpx flow-bin check or pnpm run flow should succeed

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lexical Ready Ready Preview, Comment Mar 23, 2026 3:59pm
lexical-playground Ready Ready Preview, Comment Mar 23, 2026 3:59pm

Request Review

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 23, 2026
@etrepum etrepum enabled auto-merge March 23, 2026 15:58
@etrepum etrepum added this pull request to the merge queue Mar 23, 2026
Merged via the queue into facebook:main with commit 9fbbbaf Mar 23, 2026
38 checks passed
@etrepum etrepum deleted the update-flow-306 branch March 23, 2026 16:13
@etrepum etrepum mentioned this pull request Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Flow type error regarding using "this" in input position

2 participants