Skip to content

doc: adding "rules of thumb" #12744

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

Closed
refack wants to merge 9 commits into from
+43 −0
Closed

doc: adding "rules of thumb" #12744

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
9a79fe6
doc: add "rules of thumb" section
refack Apr 29, 2017
953c7e0
[squash] add two missing '.'s
refack Apr 29, 2017
9ab59c1
[squash] add some line breaks, and fix a few typos
refack Apr 29, 2017
1691471
[squash] trying to form full sentences, and adding a line in the "com…
refack Apr 29, 2017
edc0b43
[squash] more explicit working or style and linters
refack Apr 29, 2017
011263e
[squash] Collapse explanations
refack Apr 29, 2017
3a9ee5b
Removing personal pronouns
refack Apr 29, 2017
fccb06a
[squash] its -> it is
refack Apr 29, 2017
3fd0eb2
[squash] typos & remove "mark as blocked"
refack May 2, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[squash] trying to form full sentences, and adding a line in the "com… 
…mit message" section
  • Loading branch information
@refack
refack committed Apr 29, 2017
commit 169147153c9f1a22ec05ad7f8cf5eaa6870a2580
15 changes: 8 additions & 7 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,14 @@ This document will guide you through the contribution process.
### Rules of thumb

1. #### Provide motivation for the change
Why will this change make the code better; does it fix a bug, is it a new
feature, etc. This should be in the commit messages as well as in the PR
description.
Try to explain why this change will make the code better. For example, does
it fix a bug, or is it a new feature, etc. This should expressed in the
commit messages as well as in the PR description.
2. #### Don't make _unnecessary_ code changes
Things that are changed because of personal preference or style, like:
renaming of variables or functions, adding or removing white spaces,
reordering lines or whole code blocks. These sort of changes should have
a good reason since they cause unnecessary
_Unnecessary_ code changes are changes made because of personal preference
or style. For example, renaming of variables or functions, adding or removing
white spaces, and reordering lines or whole code blocks. These sort of
changes should have a good reason since otherwise they cause unnecessary
["code churn"](https://blog.gitprime.com/why-code-churn-matters).
As part of the project's strategy we maintain multiple release lines, code
churn might hinder back-porting changes to other lines. Also when you
Expand Down Expand Up @@ -154,6 +154,7 @@ changed and why. Follow these guidelines when writing one:
Refs: http://eslint.org/docs/rules/space-in-parens.html
Refs: https://github.com/nodejs/node/pull/3615
```
- It's it's not a fix, you should document the motivation for the change
Copy link
Copy Markdown
Member

@Trott Trott Apr 29, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's it's -> If it's

Punctuation at the end of the change.

Avoid you.

So something like this perhaps:

If it's not a fix, document the motivation for the change.

Copy link
Copy Markdown
Contributor Author

@refack refack Apr 29, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed and removed you, but there are a ton of previously existing your in there.


A good commit log can look something like this:

Expand Down