Skip to content
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

Add various new cell validation rules #103

Merged
merged 11 commits into from
Mar 27, 2024
Merged

Add various new cell validation rules #103

merged 11 commits into from
Mar 27, 2024

Conversation

SmetDenis
Copy link
Member

A number of new cell validation rules have been added to both the JSON and PHP schema files, including checks for prime numbers, Fibonacci numbers, even and odd numbers, Roman numerals, and phone numbers among others. The validation logic for these new rules has also been added.

A number of new cell validation rules have been added to both the JSON and PHP schema files, including checks for prime numbers, Fibonacci numbers, even and odd numbers, Roman numerals, and phone numbers among others. The validation logic for these new rules has also been added.
The version of giggsey/libphonenumber-for-php-lite in composer.json has been bumped to "^8.13.33". The respective code diffs have also caused changes in composer.lock, updating references, content-hash, and dependencies' metadata.
@coveralls
Copy link

coveralls commented Mar 27, 2024

Pull Request Test Coverage Report for Build 8457417551

Details

  • 179 of 179 (100.0%) changed or added relevant lines in 12 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 96.162%

Totals Coverage Status
Change from base Build 8455492828: 0.3%
Covered Lines: 2355
Relevant Lines: 2449

💛 - Coveralls

The GitHub Actions workflow has been updated to include a project installation step before updates are run. This has been implemented using make commands, ensuring that the project is correctly installed and prepared for processing update commands.
The "giggsey/locale" package has been added as a dependency in the composer.json and composer.lock files. The "giggsey/locale" package provides locale functions required by libphonenumber-for-php, which is also a project dependency.
The Github workflow tasks have been renamed for clarity and ansi color outputs have been added to the composer commands. The respect/validation and giggsey/locale dependencies were updated and a new dependency, symfony/polyfill-mbstring, was added to composer.json.
An empty string has been added to the list of valid inputs in three test cases: IsRomanTest, IsVersionTest, and PhoneTest. Additionally, a new validation has been introduced in the PhoneTest class to check for the absence of the country code, enhancing the test coverage and ensuring accurate results.
Changed the quote style of 'case' and 'default' from double quotes to single quotes for consistency with the rest of syntax in .php-cs-fixer configuration. This change will maintain consistent code readability and styling throughout the configuration file.
Added three different platforms (linux/amd64, linux/arm64/v8, and linux/386) to the Docker build step in the main GitHub workflow. This will allow the built image to be used on a wider range of systems and architectures, enhancing the overall accessibility and usability of the software.
A release trigger event has been added to the demo.yml workflow file. This ensures the specified workflow runs whenever a new release is created.
@SmetDenis SmetDenis merged commit 246f2c5 into master Mar 27, 2024
9 checks passed
@SmetDenis SmetDenis deleted the new-rules branch March 27, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants