Skip to content

Remove sketchy-null lint disable (DOMRectReadOnly.js) (#59763) - #59763

Merged
meta-codesync[bot] merged 1 commit into
react:mainfrom
jcperez-ch:export-D128806044
Nov 2, 2026
Merged

meta-codesync[bot] merged 1 commit into
react:mainfrom
jcperez-ch:export-D128806044

Conversation

@jcperez-ch

@jcperez-ch jcperez-ch commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Drop sketchy-null:off from the file-level flowlint directive in DOMRectReadOnly.js, keeping unsafe-getters-setters:off. No code in the file triggers a sketchy-null lint (Flow reports nothing with the disable removed), so only the directive changes and runtime behavior is identical.

Changelog: [Internal]

Differential Revision: D128806044

@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 Nov 2, 2026
@meta-codesync

meta-codesync Bot commented Nov 2, 2026

Copy link
Copy Markdown

@jcperez-ch has exported this pull request. If you are a Meta employee, you can view the originating Diff in D128806044.

Summary:
Pull Request resolved: react#59763

Drop `sketchy-null:off` from the file-level `flowlint` directive in `DOMRectReadOnly.js`, keeping `unsafe-getters-setters:off`. No code in the file triggers a sketchy-null lint (Flow reports nothing with the disable removed), so only the directive changes and runtime behavior is identical.

Changelog: [Internal]

Differential Revision: D128806044
@meta-codesync meta-codesync Bot changed the title Remove sketchy-null lint disable (DOMRectReadOnly.js) Remove sketchy-null lint disable (DOMRectReadOnly.js) (#59763) Nov 2, 2026
@meta-codesync
meta-codesync Bot merged commit 6f05391 into react:main Nov 2, 2026
97 checks passed
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @jcperez-ch in 6f05391

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Nov 2, 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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants