Open joshualeejunyi opened 7 months ago
No details provided by team.
Team chose [severity.Low
]
Originally [severity.Medium
]
Reason for disagreement: Sequence diagrams are meant to help future developers have a clearer understanding of the project, and thus I believe this should still be Medium as it sort of defeats the point of the sequence diagrams.
Description: The sequence diagram on page 4 of the Developer Guide does not flow well and would confuse developers who are reading this for the first time.
As seen in the screenshot below,
Parser -> Command, 4 Execute
, I assume theParser
calls the execute method inCommand
, however, it shows thatCommand->Storage 5 Save Logs to File
afterwards.The next line is then
Parser->Scanner
for the input, and does not show if theStorage
returns toCommand
, which then returns to theParser
.Editing to add the second diagram below, under
Log Package
, which is very hard to follow.Screenshot: