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

Added Emojis to README #28

Closed
wants to merge 0 commits into from
Closed

Added Emojis to README #28

wants to merge 0 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 3, 2017

I've tried to add the emojis where possible. Maybe a bit too much 😁 . Hope you guys like it.

Copy link
Member

@ananyo2012 ananyo2012 left a comment

Choose a reason for hiding this comment

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

Hi @novice-97 This looks great. Thanks for your contribution. I have left some reviews. Please have a look at them and fix them. Then we will be ready to merge them!

README.md Outdated
@@ -1,13 +1,13 @@
# Contribute[![Build Status](https://api.travis-ci.org/NIT-dgp/contribute.svg?branch=master)]

Have you been looking for open source repositories to contribute? Look no further! You have come to right place. This gem will find active repositories from github to eat away your weekend time. Say goodbay to fruitless late night research, tiring click surfing and the voice in your head which make you feel guilty for not doing enough. Be warned! Your life will never be same after you have installed this gem. We offer all these unbelievbale features at a never before price of FREE! FREE! FREE!
Why are you still waiting? Get your terminals rolling, install this gem and experience the change.
Have you been looking for open source repositories to contribute :smile:?Look no further :wink:! You have come to right place. This :small_blue_diamond: gem will find active repositories from github to eat away your weekend time:clock2:. Say goodbye :smiley: to fruitless late night research :weary:, tiring click surfing and the voice in your head which make you feel guilty for not doing enough. Be warned :exclamation:Your life will never be same after you have installed this gem. We offer all these unbelievbale features at a never before price of:tada: :sparkles: FREE! FREE! FREE! :sparkles:
Copy link
Member

Choose a reason for hiding this comment

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

This line has way too much emojis. Remove some of them. The whole point of adding emojis is to make the readme interesting not to clutter the reading. So use it meaningfully. Like in headings or pointing to something.

Copy link
Author

Choose a reason for hiding this comment

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

Ok @ananyo2012 The Readme was written in such a way that it felt like it could use them all.
But i guess you are correct, so i'll do as you say.

Also i made another commit and PR on #20
before i saw this review so i'll make the changes in the next commit

Thanks for reviewing 😄

@ghost ghost closed this Aug 5, 2017
This pull request was closed.
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.

1 participant