-
Notifications
You must be signed in to change notification settings - Fork 12
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 contributing.md file #30
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.
Please update your pull request description and add the link to the issue you're trying to solve. Also, go through the review and fix the pull request.
contributing.md
Outdated
@@ -0,0 +1,140 @@ | |||
Please read through it carefully, it helps both the contributors (you guys) and maintainers be more efficient. |
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.
you guys -> you
contributing.md
Outdated
To get assigned to an issue, you'll first have to invited by us to the organization! | ||
You'll then receive an email. | ||
Once the invite if accepted, only then we'd be able to assign you to the issue. | ||
To obtain an invite check out [https://github.com/wtfpython-web/wtfpython-web/issues/11](https://github.com/wtfpython-web/wtfpython-web/issues/11) |
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.
Automated invite link has been activated and the URL is https://wtfpython-web-invite.herokuapp.com/ so you'll have to rephrase this sentence with the new link
pull_request_template.md
Outdated
@@ -0,0 +1,12 @@ | |||
NOTE: |
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.
This file doesn't belong to this pull request since it's not solving the issue #3. Please remove it.
@Pallavi024 please remove |
NOTE:
Please fill out the template below
Did you read the instructions in the WORKFLOW ?
Is your Python code styled according to the PEP8 code style convention as mentioned here ?
Have you run the tests ?
Have you followed commit guidelines as mentioned here ?