Skip to content

[Chore] Update flow suppressions to use error code#7770

Merged
takuyakanbr merged 4 commits intofacebook:mainfrom
SamChou19815:SamChou19815-patch-1
Aug 25, 2025
Merged

[Chore] Update flow suppressions to use error code#7770
takuyakanbr merged 4 commits intofacebook:mainfrom
SamChou19815:SamChou19815-patch-1

Conversation

@SamChou19815
Copy link
Copy Markdown
Contributor

Description

Flow will ban codeless suppressions soon. This PR adds error codes to a few codeless suppressions in lexical.

Test plan

flow

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 22, 2025

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

Project Deployment Preview Comments Updated (UTC)
lexical Ready Ready Preview Comment Aug 22, 2025 7:46pm
lexical-playground Ready Ready Preview Comment Aug 22, 2025 7:46pm

@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 Aug 22, 2025
declare export class AutoLinkNode extends LinkNode {
static getType(): string;
// $FlowFixMe clone method inheritance
// $FlowFixMe[incompatible-type] clone method inheritance
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot extend `LinkNode` [1] with `AutoLinkNode` because `AutoLinkNode` [2] is incompatible with `LinkNode` [3] in the
first parameter of property `clone`. [incompatible-extend]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

incompatible-type is the error code on latest version of flow with experimental.error_code_migration=new (and will be the default in the next version), which is what's used internally.

Revert the change for now, since the version used in the repo is too old. I will let the maintainer deal with this.

@takuyakanbr takuyakanbr added the extended-tests Run extended e2e tests on a PR label Aug 25, 2025
@takuyakanbr takuyakanbr added this pull request to the merge queue Aug 25, 2025
Merged via the queue into facebook:main with commit 9028e41 Aug 25, 2025
97 of 100 checks passed
@etrepum etrepum mentioned this pull request Sep 5, 2025
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. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants