-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Update README.md #223
Update README.md #223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there, looks like you've not fetched the latest changes, please fetch them and in that change, I already am covering a decent part of readme by a card. You can update that section to be not just about admin, but something like merge of about me and feedback /suggestions.My email and Linkedin if it needs to be mentioned.
Also, this pr doesn't solve the issues as you mentioned in your issue like "The new features to be added on point is missing and contribution guidelines could be made attractive and point wise" as you said.
Ok definately I will work on it |
as per your I have committed the changes, please could you merge my pr? |
I'm traveling right now and will look into asap but before that please correct the spelling mistakes in latest commit |
Ok definately |
done as per you said , now you can merge my PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as per your instruction i made the required changes , you can merge it now. |
closed div tags and removed extra whitespace
you didn't follow the complete instruction when I told you to fetch the latest changes. this branch was 95 commits behind.Now these changes need more fix :( . Have a look now |
I will fix them now, but please you need to learn from this mistake. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
please merge my PR under JWOC