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

the rule `no-new` and `no-unused-vars` conflicts? #362

Closed
leecade opened this issue Dec 16, 2015 · 1 comment

Comments

@leecade
Copy link

commented Dec 16, 2015

new Todo()
@yoshuawuyts

This comment has been minimized.

Copy link
Contributor

commented Dec 16, 2015

Hey there, thank for asking. Using new for side effects is an anti-pattern; consider making it a function call instead.

@dcousens dcousens added the question label Dec 16, 2015

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
3 participants
You can’t perform that action at this time.