Closed bmeneg closed 2 years ago
I approve this change. 😄
Merging #812 (c85ee89) into master (f8ed118) will increase coverage by
0.65%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #812 +/- ##
==========================================
+ Coverage 55.43% 56.09% +0.65%
==========================================
Files 77 77
Lines 5771 5771
==========================================
+ Hits 3199 3237 +38
+ Misses 2277 2241 -36
+ Partials 295 293 -2
Impacted Files | Coverage Δ | |
---|---|---|
cmd/issue.go | 68.18% <ø> (ø) |
|
cmd/mr_discussion.go | 85.07% <0.00%> (+28.35%) |
: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 54a8737...c85ee89. Read the comment docs.
I'm foreshadowing people wanting this to be configurable 😬
I'm foreshadowing people wanting this to be configurable 😬
Easy answer: alias it locally.
Hotkeys I understand people willing to configure, but command shorthand is too much. LOL.
"i". LOL
The 'issue' command is one of the high-demand commands within lab with, probably, 'mr', and enabling the alias 'i' for it seems a good approach for all lazy developers that we all are. 'mr', 'todo', 'ci', 'repo' are already shorthand themselves, adding this specific to 'issue' should not harm, even more considering we aren't creating one-letter aliases to every single command.
Signed-off-by: Bruno Meneguele bmeneg@redhat.com
Fix #810 .