search

ngnigel99 / pe

0 stars 0 forks source link

Optional parameters are not enclosed in command summary #14

Open ngnigel99 opened 2 years ago

ngnigel99 commented 2 years ago

image.png

the {} should enclose the parameters inside too. e.g. {-p pr...}

nus-pe-script commented 2 years ago

Team's Response

The content inside the braces is strictly for parameter. Consider the two examples below:

add task [TITLE] -d [DAY_OF_THE_WEEK] -p {PRIORITY}
add task [TITLE] -d [DAY_OF_THE_WEEK] {-p PRIORITY}

The first line is arguably more readable, and it should be clear enough for everyone to deduce that -p is not needed if PRIORITY is not used.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Going by your first statement, -p is considered a parameter as it's part of user input and gives meaning to whatever input follows,

Therefore, since -p is not enclosed in {}, the following should work by the documentation as only PRIORITY is optional.

image.png

However, I see the merit in only enclosing PRIORITY for readability - however, it's much clearer just to block out the entire chunk inclusive of -p, which is one of my considerations for issuing only a LOW severity.

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: Since it affects normal usage by the user, it's not VERY LOW - however, it only causes a minor inconvenience, thus LOW.