search

KN-CY / pe

0 stars 0 forks source link

Inconsistent use of ":" #28

Open KN-CY opened 1 year ago

KN-CY commented 1 year ago

Duke given with ":Duke" but parser just given as "Parser"

image.png

soc-pe-bot commented 1 year ago

Team's Response

It is a cosmetic bug.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I think that this is more than a cosmetic bug as its a lack of adherence to standard UML notation. This can bring some inconvenience to the reader as we will have to wonder if that is the name of the class or the name of the object instance and can only understand after looking at the source code. Same issue persists for other diagrams in the DG.

Hence I feel that not following a standard UML notation will warrant at least a low. Since very low is mostly reserved for very minor errors such as grammatical errors and minor typos, as stated in the course website.