feat(isIP): allow usage of options object#2089
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2089 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 105 105
Lines 2324 2324
Branches 586 587 +1
=========================================
Hits 2324 2324 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
if we rebase/merge this with #2091, the diff will be much cleaner and easier to review |
…o isIP-options-refactor
It is possible, but then it would be a PR in my fork and not this repo directly. So we'll have to deal with this workaround for now |
|
I think this should be good to go :). |
WikiRik
left a comment
There was a problem hiding this comment.
Can you undo the changes to the isAlpha tests?
Sorry, what exactly do you mean? I cannot see anything related to |
|
In the big test file you removed too much for this PR causing the codecov check to fail |
Done :). |
|
Can this be re-reviewed? |
This PR implements steps 1 and 2 of #1874 for
isIPand builds upon #2075.This PR extracts tests for
isIPinto a separate test file ⇾test/validators/isIP.test.js(inspired by #1793).I have also added three more tests to guard against other possible values for
version.Checklist