Skip to content

feat(isAlpha, isAlphanumeric): added support for Irish fadas (accents) #2281

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

Open
wants to merge 87 commits into
base: master
Choose a base branch
from

Conversation

khoait
Copy link

@khoait khoait commented Sep 6, 2023

Irish fadas reference:
https://en.wikipedia.org/wiki/Irish_orthography#Vowels

Checklist

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

@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.95%. Comparing base (b958bd7) to head (42d2012).
Report is 83 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2281   +/-   ##
=======================================
  Coverage   99.95%   99.95%           
=======================================
  Files         107      107           
  Lines        2454     2454           
  Branches      619      619           
=======================================
  Hits         2453     2453           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

ZhulinskiiDanil and others added 17 commits March 5, 2024 16:43
* chore: add additional testcases to isMailtoURI

Line 44 is only partially covered before this change

* chore: add additonal const to figure out which part is partial

* chore: add testcase for single ?

* chore: add another line to find partial coverage

* chore: remove default to

* chore: combine consts again
* Update isPassportNumber.js

Added the updated Azerbaijan passport validation:
See issue:
validatorjs#2274

* Update validators.test.js
…#2285)

* Add Esperanto to alpha.js

* Add Esperanto (eo) to README.md

* Add tests for Esperanto validation

* Fix tests

* Update Esperanto tests
* Pakistan vehicles license plate support

* Readme and test cases updated

* Update README.md
Added isAbaRouting validator, new validator. 
Check whether a string is ABA routing number for US bank account & cheque

ref: https://en.wikipedia.org/wiki/ABA_routing_transit_number
* chore: Publish to NPM with provenance

The release process in this repository is already automated via
GitHub Actions, which is a great first step toward creating trust in the
supply chain. Recently, NPM has started to support publishing with the
`--provenance` flag. This flag creates a link between the GitHub Actions
run that created the release and the final artifact on NPM. This linkage
further ensures that package installs can be traced back to a specific
code revision.

For more information on publishing with provenance, please refer to:
https://github.blog/2023-04-19-introducing-npm-package-provenance/

* chore: Use Node.js 18 for publishing to support provenance
* Modified symbolRegex in isStrongPassword to include '\'

* Modify test to check validity of strong password with \ character
* fix(isVAT): fixed KZ VAT number length check

* fix(isVAT): fixed KZ VAT number length test
weikangchia and others added 20 commits November 21, 2024 08:18
…idatorjs#2494)

* enhance isEmail to have regexp for host_whitelist

* update README.md

* refactor code

* update test name

* fix code logic for checkHost

---------

Co-authored-by: Rubin Bhandari <[email protected]>
…er (validatorjs#2500)

* feat: Validation for macedonian phone number added

* fix:refined regex for all mk numbers
* feat: isISO15924

* test: isISO15924

* docs: add isISO15924 to README

* refactor(isISO15924): update data sources

* refactor(isISO15924): update data source reference

* chore: fix linting
* refactor: allow for splitting tests to different files

* feat(isAfter): allow usage of options object

* style: make options italic

* refactor: rename test file extension to .test.js

* refactor: rename test-functions to testFunctions

* refactor: implement suggestion from validatorjs#2019 review

* refactor: remove custom repeat to use native function

* refactor: implement suggestion new Date

* Refactor isIP with options API

* Move and fix tests

* Add more tests

* Fix README

* Fix broken isIPRange

* refactor: change test files to prepare for validatorjs#1874

* Maintain legacy syntax

* Revert "Fix broken isIPRange"

This reverts commit cf9ebc7.

* Improve naming

* Fix comment text

* Reinstate test for isAlpha

---------

Co-authored-by: Rik Smale <[email protected]>
- Regex now covers both of the following patterns:
  -  Old pattern of: 2 letters, followed by 6 digits
  - New pattern of: 1 letter, 6 digits, 2 letters
add padding to the option list
update regexes to support validation with/without padding
update default options to keep the changes backward compatible
add new test to cover different scenarios
Co-authored-by: rajput-balram <[email protected]>
Co-authored-by: Rubin Bhandari <[email protected]>
* refactor: allow for splitting tests to different files

* feat(isAfter): allow usage of options object

* style: make options italic

* refactor: rename test file extension to .test.js

* refactor: rename test-functions to testFunctions

* refactor: implement suggestion from validatorjs#2019 review

* refactor: remove custom repeat to use native function

* refactor: implement suggestion new Date

* Refactor isBefore with options API

* Refactor isBefore tests

* Refactor to simplify logic

* Update README

* Refactor logic

* Improve README formatting

* Fix backwards-compat

* Remove redundant string assertion

* Fix comment

* Reinstate legacy tests

* Change arg name according to code review

* Add line break according to code review

* Revert change of simplifying toDate

* Fix whitespace issues

* Fix test

* Fix tests

* Format file for consistency with isBefore

* Remove redundant file

* Remove old tests

* Add tests for undefined args

* Remove arguments: linter error

* Improve comment

* Use recommended variable name

* Improve readme text according to code review

* Make isAfter arguments more robust

* Split test cases into given and default end date

---------

Co-authored-by: Rik Smale <[email protected]>
* 🐛 fix: Cyprus mobile phone validation

* 🧪 chore: update unit tests
…datorjs#2372)

* refactor: Refactor assertString

* refactor: Refactor assertString with unit tests

* add: Empty string unit test

* add: Null input value and more tests

* fix: Sentence correction
* fix validatorjs#2039

* remove line and add test to isIp.test.js

* change isIp

---------

Co-authored-by: shreysinha25 <[email protected]>
Co-authored-by: Rubin Bhandari <[email protected]>
WikiRik
WikiRik previously approved these changes Mar 28, 2025
Copy link
Member

@WikiRik WikiRik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, please fix the merge conflict

@WikiRik WikiRik added mc-to-land Just merge-conflict standing between the PR and landing. and removed 🕑 to-be-reviewed labels Mar 28, 2025
rubiin
rubiin previously approved these changes Mar 29, 2025
@khoait khoait dismissed stale reviews from rubiin and WikiRik via 7478af1 March 29, 2025 17:56
@khoait
Copy link
Author

khoait commented Mar 29, 2025

@rubiin @WikiRik the commits don't look right after rebasing my fork repo. do you have any idea? I'll check again, maybe I'll close this and open a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mc-to-land Just merge-conflict standing between the PR and landing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.