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.

Sign up for GitHub

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

Jump to bottom

[W7.6][W12-3]Ler Wei Sheng #195

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

[W7.6][W12-3]Ler Wei Sheng #195

wants to merge 10 commits into from

Conversation

lws803
Copy link

@lws803 lws803 commented Sep 25, 2018

Ready for review

CT15
CT15 reviewed Oct 5, 2018
View reviewed changes
Copy link

@CT15 CT15 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, try to avoid accumulating unrelated changes into a single PR.
I am too sure which one I should look at 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CT15 CT15 CT15 left review comments

@okkhoy okkhoy Awaiting requested review from okkhoy

Assignees

@CT15 CT15

Labels
team.3 tutorial.W12
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lws803 @CT15 @nus-se-pr-bot