sql-kit icon indicating copy to clipboard operation
sql-kit copied to clipboard

Allow to pass keyDecodingStrategy in first(decoding:) and all(decoding:)

Open valerianb opened this issue 3 years ago • 1 comments

When inserting, you can specify a keyEncodingStrategy:

try db.insert(into: myTable)
  .model(myModel, keyEncodingStrategy: .convertToSnakeCase)
  .run()

This change allows to pass a keyDecodingStrategy when using the convenience methods to decode models directly:

db.select()
  .from(myTable)
  .all(decoding: MyModel.self, keyDecodingStrategy: .convertFromSnakeCase)

valerianb avatar Jan 11 '23 09:01 valerianb

Codecov Report

Merging #161 (baf26a3) into main (a5c3df4) will not change coverage. The diff coverage is 0.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #161   +/-   ##
=======================================
  Coverage   69.27%   69.27%           
=======================================
  Files          94       94           
  Lines        3486     3486           
=======================================
  Hits         2415     2415           
  Misses       1071     1071           
Flag Coverage Δ
unittests 69.27% <0.00%> (ø)

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

Impacted Files Coverage Δ
Sources/SQLKit/Builders/SQLQueryFetcher.swift 28.94% <0.00%> (ø)

codecov-commenter avatar Jan 11 '23 09:01 codecov-commenter

This was addressed by #172

gwynne avatar May 17 '24 18:05 gwynne