Open pipipipi2002 opened 1 year ago
As it is a cosmetic issue, it is a bug but the severity level should be very low.
Team chose [severity.VeryLow
]
Originally [severity.High
]
Reason for disagreement: I disagree that this is merely a comestic issue as it involved the syntax that CS2113 is strictly using. Interface is one of those special UML diagram that has a specific syntax attached to it. Although I agree that it can be of a lower severity, it should be at least a medium since it is already taught in lecture and tutorial. Moreover, readers may not know whether your text or the diagram is the correct representation of the code, thus it should not be the case that there are inconsistencies in documentation. Especially developers might only skim through the object class diagram since it should tell them a basic idea of the class stucture.
Thus, severity should at least be a medium, instead of very low.
From the DG,
Export is an interface, as shown here
Interface objects are represented using the angled brackets << >> and not in italics.
An example from the text book is this.