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 image and myself for the Website. #788

Closed
wants to merge 2 commits into from
Closed

Added image and myself for the Website. #788

wants to merge 2 commits into from

Conversation

doluz
Copy link
Contributor

@doluz doluz commented Jan 2, 2017

Thank you for your pull request! Please read and understand everything below

Please acknowledge that you have read and understood each of the following before submitting your pull request.

Do not delete any text other than where you are instructed.

Maintainers: If any of these are left unchecked, do not merge.

Students: If one of them is applicable to you -- Please check it anyways. DO NOT REMOVE ANYTHING!

You may check them after opening your pull request.

You may do so by changing each [ ] to [x] Please take note of the whitespace as it matters.

  • Read and understood (see CONTRIBUTING.md)
  • Included a live link or preview screenshot
  • Images are 240 x 240 [w x h]
  • Resolved merge conflicts
  • Included a description of my change below

Things done in this Pull Request

Thank you.

@checkbot
Copy link

checkbot commented Jan 2, 2017

1 Warning
⚠️ You have more than one commit! Please squash them. If you need help, check CONTRIBUTING.md. Mentor note: If you’re going to merge this, use the Github ‘Squash and merge’ function (check mantainers.md).
1 Message
📖 Hi @sidhdharths thank you for your submission at Fossasia’s GCI 2016 website. Please correct any issues above, if any.

Generated by 🚫 danger

@doluz doluz closed this Jan 2, 2017
@abishekvashok
Copy link
Member

Hey you could have easily fixed this @sidhdharths here it was not needed to close it minor fixes were only needed bro I am not blaming you but please next time double check before making a PR

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.

None yet

3 participants