Skip to content

custom date coding via closures #15

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

Merged
merged 1 commit into from
Aug 14, 2025
Merged

custom date coding via closures #15

merged 1 commit into from
Aug 14, 2025

Conversation

swhitty
Copy link
Owner

@swhitty swhitty commented Aug 14, 2025

Adds custom closure based strategy to the recently added Date Encoding/Decoding strategy allowing any date format to be used.

Updates .iso8601() strategy to pass a closure that includes the ISO8601Formatter.

Catches errors from these closures and wraps in DecodingError / EncodingError.

@swhitty swhitty force-pushed the custom-date-encoding branch from cd3b703 to 5b683f6 Compare August 14, 2025 10:23
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2025

Codecov Report

❌ Patch coverage is 98.07692% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 99.80%. Comparing base (3d6ca2b) to head (9c5fc8e).

Files with missing lines Patch % Lines
Sources/KeyValueDecoder.swift 95.65% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   99.89%   99.80%   -0.10%     
==========================================
  Files           3        3              
  Lines         983     1016      +33     
==========================================
+ Hits          982     1014      +32     
- Misses          1        2       +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.

@swhitty swhitty force-pushed the custom-date-encoding branch from 5b683f6 to 9c5fc8e Compare August 14, 2025 10:25
@swhitty swhitty merged commit 7ba43c0 into main Aug 14, 2025
16 checks passed
@swhitty swhitty deleted the custom-date-encoding branch August 14, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants