albertsutz / pe

0 stars 0 forks source link

Inconsistency in Grammar #18

Open albertsutz opened 2 years ago

albertsutz commented 2 years ago

image.png

For example:

Email everyone in the student list: mail-all -> Should be Mailing everyone in the student list

Minor grammar inconsistency

nus-se-script commented 2 years ago

Team's Response

There are two bugs in this situation and we refer to the one about "closing commands".

There is a clear reason for calling it "closing commands" over "close commands" and that is to prevent ambiguity. Alternate heading suggestions could be "quit commands" or "exit commands" but the problem with all these naming suggestions is that they are extremely common and widely-used command names. Many applications use "close", "exit" or "quit" to shut their applications and we didn't want this section to come across as variations of such commands because that is not 100% accurate.

These commands would be used towards the end of one instance of TAilor running or even at the end of one semester (when you would need to reset for another set of data/block of TAilor instances). As such these two commands- together- are not doing what "close", "exit" or "quit" commands traditionally do. Therefore saying "closing commands" seemed more apt to the grouping context. These commands are used when either one run is coming to a close or one semester is coming to a close.

We would thus like to reject this bug, since we are satisfied with the naming and do not wish for it to be changed/'fixed'.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]