murtubak / pe

0 stars 0 forks source link

Method calls in code blocks could use () to indicate #12

Open murtubak opened 4 years ago

murtubak commented 4 years ago

image.png perhaps parseCommand should be parseCommand() to better indicate that it is a method instead of another class

nus-pe-bot commented 3 years ago

Team's Response

Hello, thank you for the feedback!

We have decided to reject this bug because it is clear that what we are calling is a method, for example "LogicManager calls the execute method". So we do not think that the reader will have any confusion regarding this.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]