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 #2313 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #2313 +/- ##
======================================
Coverage 70.1% 70.1%
======================================
Files 174 174
Lines 16796 16796
======================================
Hits 11775 11775
Misses 3888 3888
Partials 1133 1133
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 1371e82...b91a2f0. Read the comment docs.
Based on this discussion we first remove the unique name index on the
iterations
andareas
table before we modify the data.In a previous attempt (https://github.com/fabric8-services/fabric8-wit/pull/2312) I haven't moved the removal of the index enough up.
This should fix the migration issue that was visible here: https://github.com/openshiftio/saas-openshiftio/pull/1082