Skip to content

Update eslint-plugin-ft-flow to fix ESLint v9 issue - #55628

Closed
tcK1 wants to merge 1 commit into
react:mainfrom
tcK1:update-eslint-plugin-ft-flow
Closed

tcK1 wants to merge 1 commit into
react:mainfrom
tcK1:update-eslint-plugin-ft-flow

Conversation

@tcK1

@tcK1 tcK1 commented Feb 19, 2026 •

Copy link
Copy Markdown

Summary:

When trying to use the @react-native/eslint-config with ESLint v9, we get an issue with the eslint-plugin-ft-flow:

Oops! Something went wrong! :(

ESLint: 9.39.2

TypeError: Error while loading rule 'ft-flow/define-flow-type': context.getAllComments is not a function

Looking into it, I noticed that our current version (^2.0.0) does not support ESLint v9. Only in version ^3.0.11 it show as a valid peer dependency version.

So, although we do have a flat config, we don't support ESLint v9 out of the box.

Changelog:

[General] [Fixed] - Fixed the issue when using the flat config with ESLint v9.

Test Plan:

I've made a repro repo: https://github.com/tcK1/Repro-ESLint9-RN

Updated the ESLint version to v9 and added the eslint-plugin-ft-flow version in the resolutions configuration in the package.json.

In this state, running yarn eslint . works as expected. To reproduce the issue, remove the resolutions field (and run yarn install again to update the dependencies) and run the command again.

@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 Feb 19, 2026
@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Feb 19, 2026
@EmilJunker

Copy link
Copy Markdown

Please merge, I need this fix 🙏

@DHRUV-1000101

DHRUV-1000101 commented May 28, 2026 •

Copy link
Copy Markdown

@cortinico please see if it can be merged?

@tcK1

tcK1 commented Sep 21, 2026

Copy link
Copy Markdown
Author

Hey! Friendly reminder of this issue!

@meta-codesync

meta-codesync Bot commented Sep 21, 2026

Copy link
Copy Markdown

@cortinico has imported this pull request. If you are a Meta employee, you can view this in D121042705.

@meta-codesync meta-codesync Bot closed this in 099bcb4 Sep 22, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 22, 2026
@meta-codesync

meta-codesync Bot commented Sep 22, 2026

Copy link
Copy Markdown

@cortinico merged this pull request in 099bcb4.

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. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants