Skip to content

[T2A4][W10-A2]Lee Yi Min #128

Open
leeyimin wants to merge 7 commits intonus-cs2103-AY1617S1:masterfrom
leeyimin:T2A4
Open

[T2A4][W10-A2]Lee Yi Min #128
leeyimin wants to merge 7 commits intonus-cs2103-AY1617S1:masterfrom
leeyimin:T2A4

Conversation

@leeyimin
Copy link

@leeyimin leeyimin commented Aug 23, 2016

Ready for review

@jeffryhartanto
Copy link

I think you put the wrong team ID. [T09-A2] means Thursday 9AM team A2.

@mauris
Copy link

mauris commented Aug 28, 2016

Hi @leeyimin, your pull request title is invalid. It should be in the format of [Activity ID][Team ID] Your name, where [Activity Id] has no dashes or spaces (e.g. [T2A3] stands for Tutorial 2 Activity 3) and [Team ID] has one dash only and no spaces (e.g. [W14-A2] means Wednesday 2pm (14 hrs), Phase A, Team 2). Please follow the instructions given strictly and edit your title for reprocessing.

Note that this comment is posted by a bot sorting all the pull request submissions.

@mauris mauris changed the title [T2A4][T2-A2]Lee Yi Min [T2A4][W102-A2]Lee Yi Min Aug 28, 2016
@mauris mauris changed the title [T2A4][W102-A2]Lee Yi Min [T2A4][W10-A2]Lee Yi Min Aug 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants