search

phalcon / migrations

Generate or migrate database changes via migrations.
https://docs.phalcon.io/latest/en/db-migrations
BSD 3-Clause "New" or "Revised" License
27 stars 23 forks source link

#22 - Fix unnecessary table alter due DATE and DATETIME columns #23

Closed Jeckerson closed 4 years ago

Jeckerson commented 4 years ago

Issue - #22

sergeyklay commented 4 years ago

@Jeckerson There are some Pslam errors. Did you see?

Jeckerson commented 4 years ago

Working on them. Strange why, but there are none locally...

codecov[bot] commented 4 years ago

Codecov Report

Merging #23 into master will increase coverage by 13.12%. The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master      #23       +/-   ##
=============================================
+ Coverage     16.51%   29.64%   +13.12%     
- Complexity      600      601        +1     
=============================================
  Files            26       26               
  Lines          1580     1582        +2     
=============================================
+ Hits            261      469      +208     
+ Misses         1319     1113      -206
Impacted Files Coverage Δ Complexity Δ
src/Mvc/Model/Migration.php 39.32% <100%> (+13.73%) 161 <0> (+1) :arrow_up:
src/Migrations.php 51.37% <100%> (+36.34%) 114 <0> (ø) :arrow_down:
src/Version/IncrementalItem.php 31.76% <0%> (+7.05%) 32% <0%> (ø) :arrow_down:
src/Version/ItemCollection.php 76.59% <0%> (+63.82%) 21% <0%> (ø) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update cdab8a7...2123242. Read the comment docs.