search

nuyer / pe

0 stars 0 forks source link

Architecture diagram: No arrows pointing to Commonss #6

Open nuyer opened 6 months ago

nuyer commented 6 months ago

common no arrow.png

Commons is used by other components but there is no arrows/ dash arrows

nus-pe-script commented 6 months ago

Team's Response

Quoting the textbook

image.png

There is no requirement for all dependencies to be captured by the architecture diagram. Especially in our case, omitting the dependencies arrows between the commons component to all other classes (since they all use utility class from commons) makes the diagram clearer.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]