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

Dovansan #836

Closed
wants to merge 3 commits into from
Closed

Dovansan #836

wants to merge 3 commits into from

Conversation

dovjay
Copy link
Member

@dovjay dovjay commented Jan 5, 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 not 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

  • Add my picture to the student section
  • Add my info on student .yml

@Elgayar
Copy link
Member

Elgayar commented Jan 5, 2017

Please resolve the conflicts so that we can merge :) @dovansan

@Elgayar
Copy link
Member

Elgayar commented Jan 5, 2017

You can join the slack channel for help!

@robbyoconnor
Copy link
Contributor

There is no reason for two pull requests

@dovjay
Copy link
Member Author

dovjay commented Jan 6, 2017 via email

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.

3 participants