search

kangqiao322 / pe

0 stars 0 forks source link

Warning for invalid index can be better #9

Open kangqiao322 opened 1 year ago

kangqiao322 commented 1 year ago

image.png

The command format itself is correct, just that the index is invalid, perhaps a better warning that the index is wrong can be thrown instead.

Steps to reproduce:

Step 1: Type: "assign 0 n/p2math"

soc-se-bot commented 1 year ago

Team's Response

We already specified in the UG that the index must be a positive integer.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]