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

Revised error handling in Encodable conformance for Fields #538

Merged
merged 2 commits into from
Oct 28, 2022

Conversation

gwynne
Copy link
Member

@gwynne gwynne commented Oct 26, 2022

This allows the fixes in vapor/postgres-kit#234 and vapor/mysql-kit#313 to work correctly and adds testing of the affected functionality.

… recovery strategy used by the fixes in the database driver layers works. Includes test for the issue which triggered said fixes.
@gwynne gwynne added the semver-patch Internal changes only label Oct 26, 2022
@gwynne gwynne requested a review from 0xTim October 26, 2022 22:14
@gwynne gwynne self-assigned this Oct 26, 2022
@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2022

Codecov Report

Merging #538 (2b4dd40) into main (38670d2) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #538   +/-   ##
=======================================
  Coverage   45.72%   45.72%           
=======================================
  Files         101      101           
  Lines        5737     5737           
=======================================
  Hits         2623     2623           
  Misses       3114     3114           
Flag Coverage Δ
unittests 45.72% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
Sources/FluentKit/Model/Fields+Codable.swift 79.54% <100.00%> (ø)

@gwynne gwynne merged commit 33d498d into main Oct 28, 2022
@gwynne gwynne deleted the revised-fields-codable-error-handling branch October 28, 2022 17:06
@VaporBot
Copy link

These changes are now available in 1.35.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch Internal changes only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants