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.
Merging #2319 into master will increase coverage by
0.03%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #2319 +/- ##
==========================================
+ Coverage 69.99% 70.03% +0.03%
==========================================
Files 171 171
Lines 16610 16610
==========================================
+ Hits 11626 11632 +6
+ Misses 3861 3854 -7
- Partials 1123 1124 +1
Impacted Files | Coverage Δ | |
---|---|---|
workitem/workitem_repository.go | 67.79% <0%> (-0.23%) |
:arrow_down: |
remoteworkitem/scheduler.go | 60.97% <0%> (+7.31%) |
:arrow_up: |
remoteworkitem/jira.go | 100% <0%> (+25%) |
: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 37a913a...19d48f5. Read the comment docs.
/ok-without-tests
Use Postgres CTE to update iteration numbers
When explaining the updae before it looks like this:
Now the update looks better (notice absence of nested seq scans):