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(eslint-plugin): support BigInt in restrict-plus-operands rule #310
Conversation
column: 11, | ||
}, | ||
], | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add tests for var foo = 1n + "1"
and var foo = "1" + 1n
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@j-f1 , what do we want to get in these cases, notBigInts
or notStrings
errors?
Because now they are covered by notStrings
check, that's why I haven't added them.
Nice, thanks @webschik! |
Codecov Report
@@ Coverage Diff @@
## master #310 +/- ##
==========================================
+ Coverage 97.26% 97.27% +<.01%
==========================================
Files 67 67
Lines 2344 2348 +4
Branches 334 335 +1
==========================================
+ Hits 2280 2284 +4
Misses 43 43
Partials 21 21
|
Fixes #309