Skip to content
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

CONTRIBUTING.md: add Github contribution guidelines #4931

Merged
merged 1 commit into from
Mar 20, 2018

Conversation

jsquyres
Copy link
Member

We have a small number of requirements for contributions (e.g.,
"Signed-off-by"), so let's make sure that people have an easy way of
knowing these things.

Signed-off-by: Jeff Squyres jsquyres@cisco.com

CONTRIBUTING.md Outdated

If you have not already done so, please ensure that:

1. Every commit contains exactly this token. You can add this token
Copy link
Member

Choose a reason for hiding this comment

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

Change this token to the "Signed-off-by" token

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

We have a small number of requirements for contributions (e.g.,
"Signed-off-by"), so let's make sure that people have an easy way of
knowing these things.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
@jsquyres jsquyres force-pushed the pr/contributing-guideliens branch from 9949002 to ed3aa63 Compare March 20, 2018 14:10
@jsquyres jsquyres merged commit 5ad7962 into open-mpi:master Mar 20, 2018
@jsquyres jsquyres deleted the pr/contributing-guideliens branch March 20, 2018 21:49
@rhc54
Copy link
Contributor

rhc54 commented Mar 21, 2018

@jsquyres Something didn't work right - I'm not seeing the contribution file link when opening an issue or PR.

@jjhursey
Copy link
Member

Bonk - look in .github/CONTRIBUTING.md

@jsquyres
Copy link
Member Author

Just to follow up on this PR: it is working. But Github has lame-ified the link. 😦

github-contributing-link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants