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] Detect JSX returned by sequential expression #2801

Merged
merged 1 commit into from Sep 20, 2020

Conversation

@mikol
Copy link
Contributor

@mikol mikol commented Sep 20, 2020

Resolves #2800. Implements isReturnsSequentialJSX() and uses it in isReturningJSX() (in conjunction with isReturnsConditionalJSX() and isReturnsLogicalJSX()).

@mikol mikol force-pushed the mikol:mikol/2800/returns-sequential-jsx branch from 866f20e to c106582 Sep 20, 2020
Resolves #2800. Implements `isReturnsSequentialJSX()` and uses it in
`isReturningJSX()` (in conjunction with `isReturnsConditionalJSX()` and
`isReturnsLogicalJSX()`).
@mikol mikol force-pushed the mikol:mikol/2800/returns-sequential-jsx branch from c106582 to dec2975 Sep 20, 2020
@ljharb
ljharb approved these changes Sep 20, 2020
@ljharb ljharb force-pushed the mikol:mikol/2800/returns-sequential-jsx branch 2 times, most recently from 5f9876e to 74c4eed Sep 20, 2020
@ljharb ljharb merged commit 74c4eed into yannickcr:master Sep 20, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.001%) to 97.496%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.