Skip to content
This repository has been archived by the owner on Aug 4, 2020. It is now read-only.

Support optional calls in no-unused-expressions #158

Conversation

lehni
Copy link
Contributor

@lehni lehni commented Sep 3, 2018

This relates to the issue outlined here: babel/babel-eslint#643

@lehni lehni force-pushed the fix/optional-call-expression-in-no-unused-expressions branch from d0f7952 to b89ec98 Compare September 3, 2018 21:02
Copy link
Member

@kaicataldo kaicataldo left a comment

Choose a reason for hiding this comment

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

Noticed a few things while taking a quick glance at this

@lehni
Copy link
Contributor Author

lehni commented Sep 4, 2018

@kaicataldo I've addressed the things in the feedback now. Please take a look.

@lehni
Copy link
Contributor Author

lehni commented Sep 5, 2018

@existentialism I've addressed all things now. Is this good to go?

@existentialism existentialism merged commit d653393 into babel:master Sep 5, 2018
@existentialism
Copy link
Member

@lehni thanks!

@lehni
Copy link
Contributor Author

lehni commented Sep 5, 2018

@existentialism great, thanks! Do you think this warrants a new release soon? Would be great!

@existentialism
Copy link
Member

@lehni published as 5.2.0!

nicolo-ribaudo pushed a commit to babel/babel that referenced this pull request Nov 14, 2019
…nal-call-expression-in-no-unused-expressions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants