Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[W3.3][F09-A2]Shi Tian #16

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[W3.3][F09-A2]Shi Tian #16

wants to merge 5 commits into from

Conversation

shitian007
Copy link

No description provided.

@nus-se-pr-bot
Copy link

nus-se-pr-bot commented Sep 8, 2017

Hi @shitian007, your pull request title is invalid.

It should be in the format of [Learning Outcome ID][Team ID] Your Name, where [Learning Outcome ID] has no dashes or spaces (e.g. [W3.1a]) 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.

Submit only one learning outcome per pull request (unless otherwise stated in instructions) and do remember to create your branches from the commit where the master branch is pointing at so that each PR you submit only consist of commits meant for the activity.

Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at cs2103-pr-bot and add a link to this PR.

@satyam-cyc
Copy link

Well Done! Keep up the good work.

You can expect more rigorous reviews in your upcoming PRs.
You should now close this PR.

@shitian007 shitian007 changed the title Make case command insensitive [W3.3][F09-A2]Shi Tian Sep 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants