What kind of change does this PR introduce? (check at least one)
[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:
Does this PR introduce a breaking change? (check one)
[x] Yes
[ ] No
If yes, please describe the impact and migration path for existing applications:
Currently sdq.isCedula('40222075711') == sdq.isCedula('402a2@2-075/7_1 1') but in the case of having any non-digit character should return false.
You will have to replace all non-digit character before passing to the isCedula or IsRnc function.
The PR fulfills these requirements:
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing
[x] New/updated tests are included
If adding a new feature, the PR's description includes:
[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
Currently sdq.isCedula('40222075711') == sdq.isCedula('402a2@2-075/7_1 1') but in the case of having any non-digit character should return false.
You will have to replace all non-digit character before passing to the isCedula or IsRnc function.
The PR fulfills these requirements:
fix #xxx[,#xxx], where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Other information: