-
-
Notifications
You must be signed in to change notification settings - Fork 745
Closed
Labels
good first issueEasier issue for first time contributorsEasier issue for first time contributors
Description
Minor change to make the code more readable. Array.includes
is available in Node 6+ and can be used to replace the less-readable [].indexOf !== -1
or [].indexOf > -1
.
In general, Array.includes
is a drop in replacement but there are some slight differences in edge cases (most notably when searching for undefined
).
https://dev.to/adroitcoder/includes-vs-indexof-in-javascript
Metadata
Metadata
Assignees
Labels
good first issueEasier issue for first time contributorsEasier issue for first time contributors