Feedback on week 6 project progress

[nus-se-bot]

Subject: feedback on week 6 project progress

See v1.1 progress guide for more details of the activities mentioned below.

Team progress

• [x] Team org and repo set up with the correct names (:heavy_check_mark: well done!)
• [x] PR created from team repo to AB-3 or AB-4 (:heavy_check_mark: well done!)

Individual progress of @beesaycheese

• Used forking workflow:
  • [x] Team repo forked to your GitHub account (:heavy_check_mark: well done!)
  • [x] PRs created to update Java code (:heavy_check_mark: well done!)
  • [ ] PRs created to update documents (:exclamation: try to do by next milestone)

Individual progress of @whyjayxp

• Used forking workflow:
  • [x] Team repo forked to your GitHub account (:heavy_check_mark: well done!)
  • [ ] PRs created to update Java code (:exclamation: try to do by next milestone)
  • [ ] PRs created to update documents (:exclamation: try to do by next milestone)

Individual progress of @qing-yuan

• Used forking workflow:
  • [x] Team repo forked to your GitHub account (:heavy_check_mark: well done!)
  • [ ] PRs created to update Java code (:exclamation: try to do by next milestone)
  • [ ] PRs created to update documents (:exclamation: try to do by next milestone)

Individual progress of @bava98

• Used forking workflow:
  • [x] Team repo forked to your GitHub account (:heavy_check_mark: well done!)
  • [x] PRs created to update Java code (:heavy_check_mark: well done!)
  • [ ] PRs created to update documents (:exclamation: try to do by next milestone)

Tutor: @marlenekoh

Note: the above observation was done by the CS2103-feedback-bot and covers changes up to 2019-02-20 00:00:00 only. If you think the above observation is incorrect, please let us know by replying in this thread. Please include links to relevant PRs/comments in your response.

[beesaycheese]

@marlenekoh Hi Marlene, we realised that the bot did not tick "PRs created to update documents". Actually, we did accomplish this milestone last week. However, we did so by editing the contents directly on to the main. This was done during one of our meetings, where all of us were present to give consent to the changes made. Hence, there was no need for us to pull request. May we, therefore, request for you to take note that we have actually finish our milestone please? Thank you!

[marlenekoh]

@beesaycheese Noted (: Will take a look, thank you!

@marlenekoh Hi Marlene, we realised that the bot did not tick "PRs created to update documents". Actually, we did accomplish this milestone last week. However, we did so by editing the contents directly on to the main. This was done during one of our meetings, where all of us were present to give consent to the changes made. Hence, there was no need for us to pull request. May we, therefore, request for you to take note that we have actually finish our milestone please? Thank you!

[okkhoy]

We want you to follow the workflow properly at least until the first couple of milestones. Hence the bot is configured to check for PRs. Since there was no PR for documentation, the bot has rightly identified it so.

No much damage done; we want you to achieve only 60% of the project management tasks, so you have a lot of leeway.

For v1.1 please ensure that you have a PR for some documentation changes.