Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

481/add native date to result #482

Merged
merged 8 commits into from
Feb 9, 2023
Merged

Conversation

MauricioRobayo
Copy link
Owner

closes #481

@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Base: 100.00% // Head: 98.75% // Decreases project coverage by -1.25% ⚠️

Coverage data is based on head (81bab58) compared to base (b81e30f).
Patch coverage: 98.18% of modified lines in pull request are covered.

❗ Current head 81bab58 differs from pull request most recent head 71a6815. Consider uploading reports for the commit 71a6815 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #482      +/-   ##
===========================================
- Coverage   100.00%   98.75%   -1.25%     
===========================================
  Files            3        5       +2     
  Lines           42       80      +38     
  Branches         6       23      +17     
===========================================
+ Hits            42       79      +37     
- Misses           0        1       +1     
Impacted Files Coverage Δ
src/holidays.ts 100.00% <ø> (ø)
src/index.ts 95.23% <92.85%> (-4.77%) ⬇️
src/helpers.ts 100.00% <100.00%> (ø)
src/utils/holidaysWithinInterval.ts 100.00% <100.00%> (ø)
src/utils/isHoliday.ts 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Owner Author

@MauricioRobayo MauricioRobayo left a comment

Choose a reason for hiding this comment

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

lgtm

@MauricioRobayo MauricioRobayo merged commit dc02e50 into main Feb 9, 2023
@MauricioRobayo MauricioRobayo deleted the 481/Add-native-date-to-result branch February 9, 2023 19:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add native date to result
1 participant