JamesLiuZX / pe

0 stars 0 forks source link

Typo in sequence diagram #17

Open JamesLiuZX opened 1 year ago

JamesLiuZX commented 1 year ago

should be executeIndices() for plural in your code and docs.

Screenshot 2023-04-14 at 3.38.59 PM.png

nus-se-script commented 1 year ago

Team's Response

Hi!

"indexes" is a widely-accepted plural form of "index". You may refer to the following links that elaborate more on this plural form:

Microsoft's Style Guide

NASDAQ

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]