Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix function type for flow #25965

Merged
merged 1 commit into from
Jan 5, 2023
Merged

fix function type for flow #25965

merged 1 commit into from
Jan 5, 2023

Conversation

mondaychen
Copy link
Contributor

These files fail CI yarn_flow.

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jan 5, 2023
Copy link
Member

@kassens kassens left a comment

Choose a reason for hiding this comment

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

Thanks!

@kassens
Copy link
Member

kassens commented Jan 5, 2023

CI breakage seems to be unrelated.

@kassens kassens merged commit 2619886 into facebook:main Jan 5, 2023
github-actions bot pushed a commit that referenced this pull request Jan 5, 2023
@@ -15,7 +15,7 @@ function generateArray(size) {

const arr = generateArray(50000);

export default function LargeSubtree() {
export default function LargeSubtree(): React.Node {
Copy link

Choose a reason for hiding this comment

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

React.Node or React.ReactNode ?

Copy link
Contributor

@ymqy ymqy Jan 6, 2023

Choose a reason for hiding this comment

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

@hock6191
Copy link

hock6191 commented Jan 6, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants