search

OrangeCatLoves / pe

0 stars 0 forks source link

Lack of alternative command #5

Open OrangeCatLoves opened 1 week ago

OrangeCatLoves commented 1 week ago

image.png

I noticed that for your enroll command, perhaps it's actually better to accept enr as well for the same behaviour. This is better for people who are much more familiar with your application, reducing the time taken for typing the commands. Typing enr is certainly faster than enroll. Having enr can facilitate fast CLI typers and people who are familiar with your application. I understand that you have an edit command so shortening the command enroll to e is not possible, enr would be optimal.

soc-pe-bot commented 5 days ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

We understand the issue raised here. Our original belief is that the word enroll should not take up much time for a fast typist, even if we change the command to enr, it is only faster by three letters. In general, we tried to keep the commands under 10 characters. We could further optimised by reducing the length of the command in the future, but we believe it is not the most important feature that we should have now, so it will classified as notInScope

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]