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.

Sign up for GitHub

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

Jump to bottom

Updated screenshot #30

Closed
wants to merge 1 commit into from
Closed

Updated screenshot #30

wants to merge 1 commit into from

Conversation

Aman8817
Copy link
Contributor

@Aman8817 Aman8817 commented Dec 6, 2018

All Submissions:

  • Have you followed the guidelines in our Contributing document?

New Feature Submissions:

  1. Does your submission pass build tests locally?
  2. Have you indented your code locally prior to submission?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully ran tests with your changes locally?

@monsij
Copy link
Member

monsij commented Dec 6, 2018

Hope that you have followed the contribution guidelines :-)

(Powered by GitMate.io)

@Aman8817
Copy link
Contributor Author

Aman8817 commented Dec 6, 2018

I have done work for #26

@monsij
Copy link
Member

monsij commented Dec 7, 2018

Please make a PR from a branch of your fork @Aman8817

@monsij
Copy link
Member

monsij commented Dec 8, 2018

unack bfa62d7

@monsij monsij closed this Dec 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harry-hov harry-hov harry-hov approved these changes

Assignees

@Aman8817 Aman8817

Labels
process/pending_review size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Aman8817 @monsij @harry-hov