Skip to content

Pull Request Info - Please read!#186

Open
emilyparkes wants to merge 8 commits intodevfrom
issue-pull-request-info
Open

Pull Request Info - Please read!#186
emilyparkes wants to merge 8 commits intodevfrom
issue-pull-request-info

Conversation

@emilyparkes
Copy link
Collaborator

@emilyparkes emilyparkes commented Apr 25, 2018

Hey there,
Just to make it easy as this project gets bigger, please could you do the following:

Pull Request Name
The same name as the issue it refers to, or a short description with more detail of the changes in the comments.

Reviewers
Add Mārama-App/devs as the reviewer.

Assignees
Assign yourself to the PR.

Labels
Add any labels that apply to your issue.

Projects
Add Mārama as the project.

Milestone
If applicable, add the milestone that is on your issue, or that applies to your PR.

Reference
Please add a comment that references your issue like so: ISSUE #188

This way it is very easy to find any changes to code, what section what changed, what issue you were working on and what pull request it belongs to (& vice versa).

In future, any contributors will also follow this system - most other big repo's do this.
Thanks!

DO NOT MERGE THIS REQUEST

@emilyparkes emilyparkes changed the title Issue & Pull Request Info - Please read. Pull Request Info - Please read! May 2, 2018
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.

1 participant