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
Update contributing guidelines #329
Conversation
|
||
Once the changes are made locally, the author creates a new commit on their | ||
newly created branch based on master, then pushes to their own fork, and creates | ||
a PR from the GitHub UI. The author then selects one or more reviewers. |
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.
Might be a bit tricky for an external contributor to select a reviewer?
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.
Yeah, hopefully some of us will notice though.
CONTRIBUTING.md
Outdated
a PR from the GitHub UI. The author then selects one or more reviewers. | ||
|
||
The commit / PR description should follow | ||
[standard git commit conventions](https://chris.beams.io/posts/git-commit/). |
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.
nit: s/git/Git/
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.
Done.
CONTRIBUTING.md
Outdated
[standard git commit conventions](https://chris.beams.io/posts/git-commit/). | ||
button in the GitHub UI. The UI will suggest a final commit message composed of | ||
the PR title, and individual commit messages as a bullet point list; the author | ||
can then reword the final commit message in the UI, usually discarding the |
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.
s/can/should/
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.
Done.
No description provided.