| |
@@ -209,13 +209,12 @@
|
| |
release = int(get_response(prompt, helptext, accepted=validrels))
|
| |
|
| |
while not version_set:
|
| |
- if not version_type:
|
| |
- prompt = (
|
| |
- "Would you like to report a result for a milestone "
|
| |
- "(c)ompose or a (n)ightly compose? "
|
| |
- )
|
| |
- helptext = "Please choose (c)ompose or (n)ightly."
|
| |
- version_type = get_response(prompt, helptext, accepted=("c", "n"))
|
| |
+ prompt = (
|
| |
+ "Would you like to report a result for a milestone "
|
| |
+ "(c)ompose or a (n)ightly compose? "
|
| |
+ )
|
| |
+ helptext = "Please choose (c)ompose or (n)ightly."
|
| |
+ version_type = get_response(prompt, helptext, accepted=("c", "n"))
|
| |
|
| |
if version_type == "c":
|
| |
prompt = "(B)eta\n(R)C\nWhich milestone would you like to report a milestone for? "
|
| |
This is a followup from #20.
Cleanup unreachble code for coverage reasons.
Remove
if not version_type
conditional:version_type
is explicitly set to '', andnot ''
will always evaluate toTrue
.Remove elif in
check
:A
ValueError
is raised if bothaccepted
andaccfunc
areNone
, so theelif
was not necessary.