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

Add CONTRIBUTING.md #196

Merged
merged 1 commit into from Oct 6, 2015
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
39 changes: 39 additions & 0 deletions CONTRIBUTING.md
@@ -0,0 +1,39 @@
# Contributing
## Issues
When filing an issue, make sure to answer these five questions:

1. What version of the project are you using?
2. What operating system and processor architecture are you using?
3. What did you do?
4. What did you expect to see?
5. What did you see instead?

## Code
### Proposals
Non trivial changes should be first discussed with the project maintainers by
opening a Github issue with the "Proposal: " title prefix, clearly explaining
rationale, context and implementation ideas.

This separate step is encouraged but not required.

### Implementation
Work should happen in an open pull request having a WIP prefix in its
title which gives visibility to the development process and provides
continuous integration feedback.

The pull request description must be well written and provide the necessary
context and background for review. If there's a proposal issue, it must be
referenced. When ready, replace the WIP prefix with PTAL which will
bring your contribution to the attention of project maintainers who will review
your PR in a timely manner.

Before review, keep in mind that:
- Git commit messages should conform to [community standards](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
- Git commits should represent meaningful milestones or units of work.
- Changed or added code must be well tested. Different kinds of code
require different testing strategies.
- Changed or added code must pass the project's CI.
- Changes to vendored files must be grouped into a single commit.

Once comments and revisions on the implementation wind down, the reviewers will
add the LGTM label which marks the PR as merge-able.
4 changes: 4 additions & 0 deletions README.md
Expand Up @@ -45,5 +45,9 @@ $ go get github.com/mesos/mesos-go
$ go test -race ./...
```

## Contributing
Contributions are welcome. Please refer to [CONTRIBUTING.md](CONTRIBUTING.md) for
guidelines.

## License
This project is [Apache License 2.0](LICENSE).