Open billneff79 opened 5 years ago
Merging #157 into develop will decrease coverage by
1.29%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## develop #157 +/- ##
==========================================
- Coverage 86.44% 85.15% -1.3%
==========================================
Files 58 59 +1
Lines 1372 1374 +2
Branches 181 181
==========================================
- Hits 1186 1170 -16
- Misses 186 204 +18
Impacted Files | Coverage Δ | |
---|---|---|
lib/helpers/reports/forChannelUpdate.js | 33.33% <0%> (-66.67%) |
:arrow_down: |
lib/helpers/updateChannelReport.js | 19.04% <0%> (-57.15%) |
:arrow_down: |
lib/helpers/doChannelReport.js | 94.73% <0%> (-5.27%) |
:arrow_down: |
lib/helpers/doBlock.js | 91.66% <0%> (-1.67%) |
:arrow_down: |
models/index.js | 92.85% <0%> (ø) |
|
lib/helpers/reports/transformers/index.js | 100% <0%> (+7.14%) |
:arrow_up: |
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 4c8b97a...a119a49. Read the comment docs.
Due to the order that the database migrations were being run in, a brand-new database setup would fail when running
migrate-to-local-times
because the model was trying to use apostUpdatesInChannel
field that isn't present in the database yet. By changing the file names in this PR, thepostUpdatesInChannel
field gets added before it is used by`migrate-to-local-times
and the databases get setup properly now, even on a fresh install