search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Bug for redundant class in class diagram #2068

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

Documentation Bug (Severity LOW)

Would the CantVasMain class still be necessary since readers already know that that class is the main class and will always be used?

image.png

[original: nus-cs2113-AY2324S2/pe-interim#2466] [original labels: severity.Low type.DocumentationBug]

Haotian199 commented 5 months ago

Team's Response

It is true that CantVasMain is not necessary for the sequence diagram, but it does not affect the clarity and flow of logic of the sequence diagram as a whole. Therefore, it will be more fitting for the severity of this issue to be very low instead of low.

Duplicate status (if any):

--