Skip to content

docs/CONTRIBUTING: rewrite & add commit guidelines#174

Merged
shaleman merged 1 commit intocontiv:masterfrom
unclejack:rewrite_contributing
Nov 4, 2015
Merged

docs/CONTRIBUTING: rewrite & add commit guidelines#174
shaleman merged 1 commit intocontiv:masterfrom
unclejack:rewrite_contributing

Conversation

@unclejack
Copy link
Copy Markdown
Contributor

No description provided.

@mapuri
Copy link
Copy Markdown
Contributor

mapuri commented Nov 3, 2015

LGTM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe instructions to git commit -s here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@erikh: Good point, I'll add that.

@erikh
Copy link
Copy Markdown
Contributor

erikh commented Nov 3, 2015

Looks great other than that nit

@unclejack unclejack force-pushed the rewrite_contributing branch from 86402f2 to 9d5b99a Compare November 3, 2015 20:40
@unclejack
Copy link
Copy Markdown
Contributor Author

The PR has been updated to make the following changes:

  • explain that the detailed description of the change is optional and should be provided only when it's deemed necessary
  • provide instructions on how to use git commit -s and how to set up the required git config options
  • add another example of a commit with a sign off and without any detailed description

PTAL

@erikh
Copy link
Copy Markdown
Contributor

erikh commented Nov 4, 2015

LGTM

shaleman added a commit that referenced this pull request Nov 4, 2015
docs/CONTRIBUTING: rewrite & add commit guidelines
@shaleman shaleman merged commit 6764acc into contiv:master Nov 4, 2015
@unclejack unclejack deleted the rewrite_contributing branch November 9, 2015 14:48
@unclejack unclejack restored the rewrite_contributing branch November 10, 2015 11:07
@unclejack unclejack deleted the rewrite_contributing branch January 6, 2018 12:30
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.

5 participants