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

Add ChuaJunWen-Intro.md #549 #550

Merged

Conversation

JunWen991
Copy link

No description provided.

@JunWen991
Copy link
Author

Fixes #549

@JunWen991
Copy link
Author

Ready for review

@xdshang
Copy link

xdshang commented Aug 25, 2016

please remove the spaces in {YourName}.txt, to keep consistency with {YourName}-Intro.md.

@JunWen991 JunWen991 changed the title Add Chua Jun Wen-Intro.md #549 Add ChuaJunWen-Intro.md #549 Aug 25, 2016
@xdshang
Copy link

xdshang commented Aug 25, 2016

remove the wrongly named file.

@JunWen991
Copy link
Author

Changed Chua Jun Wen.txt to ChuaJunWen.txt

Ready for Review

@xdshang
Copy link

xdshang commented Aug 25, 2016

Chua Jun Wen.txt is no longer needed, please delete it.

@JunWen991
Copy link
Author

Deleted.
Push, Everything Up-To-Date
Ready for Review.

@xdshang
Copy link

xdshang commented Aug 25, 2016

Did you deleted the file from this pr?

@JunWen991
Copy link
Author

I'm sorry, how do i delete that file from this PR?
I'm new to this GitHub. Thanks

@xdshang
Copy link

xdshang commented Aug 25, 2016

delete the file in your local repo -> commit the changes -> push to your remote repo
this is a rough flow, I think.

@JunWen991
Copy link
Author

Deleted Chua Jun Wen.txt

Ready for review.

@xdshang xdshang merged commit f08e333 into nus-cs2103-AY1617S1:master Aug 25, 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.

2 participants