Closed prarit closed 2 years ago
@zampierilucas @bmeneg , I could make this "lab todo clean" instead of "lab todo list --clean". I cannot decide if this functionality should be its own call or not.
Merging #754 (5b32dcf) into master (5ee7e23) will decrease coverage by
0.07%
. The diff coverage is10.00%
.
@@ Coverage Diff @@
## master #754 +/- ##
==========================================
- Coverage 54.82% 54.74% -0.08%
==========================================
Files 77 77
Lines 5611 5621 +10
==========================================
+ Hits 3076 3077 +1
- Misses 2255 2264 +9
Partials 280 280
Impacted Files | Coverage Δ | |
---|---|---|
cmd/todo_list.go | 12.08% <10.00%> (-0.26%) |
: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 5ee7e23...5b32dcf. Read the comment docs.
@prarit, isn't this functionality the same as the one implemented by lab todo done --all
?
Oh .. duh. If only I knew the person who wrote that other command.
I use my Todo list to keep track of important work and find myself being lazy about keeping my Todo list clean. I often end up with a large number of closed or merged MRs on my Todo list, and it would be useful to have an lab option to remove these Todos from my Todo list.
Add a clean option, "lab todo list --clean" that removes Todos for closed or merged MRs.
Signed-off-by: Prarit Bhargava prarit@redhat.com