Closed kwk closed 5 years ago
Thank you @kwk for this contribution!
It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests
as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.
/ok-without-tests
Merging #2325 into master will decrease coverage by
0.01%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #2325 +/- ##
==========================================
- Coverage 70.04% 70.02% -0.02%
==========================================
Files 171 171
Lines 16610 16609 -1
==========================================
- Hits 11635 11631 -4
- Misses 3853 3854 +1
- Partials 1122 1124 +2
Impacted Files | Coverage Δ | |
---|---|---|
migration/migration.go | 69.17% <ø> (-0.12%) |
:arrow_down: |
controller/workitem.go | 78.27% <0%> (-0.57%) |
: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 7fda6af...03cd564. Read the comment docs.
It was postponed to #2326
Currently the database migration 110 won't go through on prod-preview. I'm postponing it for now.