Tagging earlier commits with deliverables and naming of classes

[limhawjia]

Hi, I will be posting Prof Damith's response here for those who are facing the same issue.

1. For some of the deliverables, e.g. A-MoreOOP, we are required to instructed to construct certain classes with specific names to do specific tasks. However, if we have already implemented classes with the same functionality beforehand, do we have to change their names to match those in the instructions?

It's fine. But you can consider renaming the classes to match the instructions unless you have strong reasons to keep the current name. Using the specified names can make the code reviewers' job easier later.

2. Also, for deliverables that we have already implemented prior to its corresponding week, e.g. A-Packages in my case, can we simply tag a previous commit with the deliverable?

Yes.

[damithc]

Thanks for sharing this @limhawjia