Skip to content

Conversation

@NE-SmallTown
Copy link
Contributor

@NE-SmallTown NE-SmallTown commented Sep 7, 2019

Seems this test case copies from the former one which has a { this.state.active ? <button</button> : null}, but this test case doesn't need that

@sizebot
Copy link

sizebot commented Sep 7, 2019

No significant bundle size changes to report.

Generated by 🚫 dangerJS against dd337d5

Copy link
Contributor

@threepointone threepointone left a comment

Choose a reason for hiding this comment

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

Nice catch! Thanks.

@threepointone threepointone merged commit 9444c87 into facebook:master Sep 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants