vapor / fluent-kit

Swift ORM (queries, models, and relations) for NoSQL and SQL databases
MIT License
217 stars 116 forks source link

Log start and finish of migration prepares and reverts #526

Closed gwynne closed 2 years ago

gwynne commented 2 years ago

Also adds some documentation comments to the confusion that is Property.swift.

Shoutout to @dmonagle for being the first to point out to me that this logging was missing!

codecov-commenter commented 2 years ago

Codecov Report

Merging #526 (79fcf44) into main (64f799f) will increase coverage by 2.87%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
+ Coverage   42.03%   44.91%   +2.87%     
==========================================
  Files         112      101      -11     
  Lines        6280     5769     -511     
==========================================
- Hits         2640     2591      -49     
+ Misses       3640     3178     -462     
Flag Coverage Δ
unittests 44.91% <0.00%> (+2.87%) :arrow_up:

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

Impacted Files Coverage Δ
Sources/FluentKit/Migration/Migrator.swift 0.00% <0.00%> (ø)
Sources/FluentKit/Properties/Property.swift 75.00% <ø> (ø)
...ources/FluentBenchmark/SolarSystem/PlanetTag.swift
...ntBenchmark/SolarSystem/GalacticJurisdiction.swift
Sources/FluentBenchmark/SolarSystem/Galaxy.swift
Sources/FluentBenchmark/SolarSystem/Star.swift
...rces/FluentBenchmark/SolarSystem/SolarSystem.swift
Sources/FluentBenchmark/SolarSystem/Governor.swift
Sources/FluentBenchmark/SolarSystem/Tag.swift
...ces/FluentBenchmark/SolarSystem/Jurisdiction.swift
... and 3 more
VaporBot commented 2 years ago

These changes are now available in 1.33.3