Added countries to isPassportNumber.

[JuanFML]

Enlarged feat isPassportNumber #1288 : Added more countries into the list

Added the next countries with their corresponding tests:

• Mexico, MX
• Kazakhstan, KZ
• Liechtenstein, LI
• Malaysia, MY
• Thailand, TH
• New Zeland, NZ
• Jamaica, JM

Checklist

• PR contains only changes related; no stray files, etc.
• README updated (where applicable)
• Tests written (where applicable)

[codecov]

Codecov Report

Merging #1555 (7529acf) into master (787df19) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #1555   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           99        99           
  Lines         1776      1776           
=========================================
  Hits          1776      1776           

Impacted Files	Coverage Δ
src/lib/isPassportNumber.js	100.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 787df19...7529acf. Read the comment docs.

[tux-tn]

Congrats @JuanFML for your first PR 🎉
LGTM !

[profnandaa]

This will be merged after our release rework PR #1553

[tux-tn]

@JuanFML Can you fix conflicts?

[profnandaa]

@JuanFML -- pls fix the merge conflicts and we should be good to go. Thanks for your contribution!

[rubiin]

@profnandaa can i work on this as author is awol

[ezkemboi]

@JuanFML can you have some time to polish these merge conflicts and we can get this merged?

@rubiin we just want to get these people credits for the work they have done instead of building it as our own.

[fedeci]

If the PR is really abandoned, we can still create a new one based on this and add @JuanFML as co-author.

[ezkemboi]

@fedeci when doing clean up of stale PR's, we don't create new nor delete the existing one.
Please check this guide on how we do clean stale Pr's

https://github.com/validatorjs/validator.js/wiki/Maintenance:-PR-Clean-Up

[fedeci]

The last step explicitly say create a new PR :)
I think we meant the same process, however.

[ezkemboi]

Cool. Noted.

…

On Fri, Oct 8, 2021, 3:00 PM Federico Ciardi ***@***.***> wrote: The last step explicitly say create a new PR :) — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1555 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALVWPH7DPVBYUWH3E5J4CLDUF3MM7ANCNFSM4U3POVVQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[rubiin]

@ezkemboi yes we can do this.

[profnandaa]

LGTM except for the merge conflicts, will try fix them.

[profnandaa]

included in the combined PR - #2164