Merrickneo / pe

0 stars 0 forks source link

Incorrect figure labelling for Parser classes for the Logic component #11

Open Merrickneo opened 1 year ago

Merrickneo commented 1 year ago

Figure label should be Class diagram for "Logic Parser"

Screenshot 2023-04-14 at 3.17.28 PM.png

soc-pe-bot commented 1 year ago

Team's Response

Logic contains the parser and other classes but the name of the logic component is actually just Logic.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I get where the team is coming from. However, since figure 1.6 above says Class Diagram for "Logic" components, this class diagram's naming made me a little confused.

As such, changing it to "Class Diagram of "Logic Parser"" would be a more appropriate description to help with the comprehensibility of the reader. As such, I would think this would be a valid issue but does not really affect the documentation as a whole.

Hence, I would say this should be under NotInScope as it is valid in some areas but does not warrant a deduction for the dev team.