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 date interval validation rules #123

Merged
merged 3 commits into from
Apr 2, 2024
Merged

Add date interval validation rules #123

merged 3 commits into from
Apr 2, 2024

Conversation

SmetDenis
Copy link
Member

Updated the data validation code to include date interval checks.

Minor changes have been implemented across several files. These include text format updates in both the composer.json and README.md files for better readability. Additionally, a comment has been added in the csv-blueprint.php to clarify the purpose of error handling.
Updated the data validation code to include date interval checks. This added multiple new lines in JSON, PHP, YML and YAML file types to define these rules. In addition, a new file implementing the date interval validation logic was created, as well as a test file to ensure this feature works as expected.
@SmetDenis SmetDenis merged commit 43f635e into master Apr 2, 2024
9 checks passed
@SmetDenis SmetDenis deleted the dateinterval branch April 2, 2024 12:05
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8522415694

Details

  • 47 of 52 (90.38%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 96.724%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Rules/Cell/ComboDateInterval.php 47 52 90.38%
Totals Coverage Status
Change from base Build 8506551283: -0.1%
Covered Lines: 2598
Relevant Lines: 2686

💛 - Coveralls

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.

None yet

2 participants