Closed jarifibrahim closed 5 years ago
Thank you @jarifibrahim 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-test
/ok-without-tests
[test]
[test]
[test]
Merging #2310 into master will decrease coverage by
0.04%
. The diff coverage is0%
.
@@ Coverage Diff @@
## master #2310 +/- ##
==========================================
- Coverage 70.13% 70.08% -0.05%
==========================================
Files 174 174
Lines 16784 16792 +8
==========================================
- Hits 11771 11769 -2
- Misses 3880 3889 +9
- Partials 1133 1134 +1
Impacted Files | Coverage Δ | |
---|---|---|
controller/space.go | 69.64% <0%> (-1.13%) |
:arrow_down: |
workitem/workitem_repository.go | 67.79% <0%> (-0.23%) |
: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 e75e0d1...7928b20. Read the comment docs.
This PR adds more logs to the delete action on space controller.
See https://github.com/fabric8-services/fabric8-wit/pull/2280 and https://github.com/fabric8-services/fabric8-wit/pull/2297