mirror of
https://github.com/prometheus/prometheus.git
synced 2024-12-25 21:54:10 -08:00
Add section for new contributors
Inspiration from this blog post: https://manishearth.github.io/blog/2016/01/03/making-your-open-source-project-newcomer-friendly/ Some parts are lifted from: https://github.com/servo/servo/blob/master/CONTRIBUTING.md Signed-off-by: Goutham Veeramachaneni <cs14btech11014@iith.ac.in>
This commit is contained in:
parent
6a767b736b
commit
d733769722
|
@ -2,6 +2,8 @@
|
||||||
|
|
||||||
Prometheus uses GitHub to manage reviews of pull requests.
|
Prometheus uses GitHub to manage reviews of pull requests.
|
||||||
|
|
||||||
|
* If you are a new contributor see: [Steps to Contribute](#steps-to-contribute)
|
||||||
|
|
||||||
* If you have a trivial fix or improvement, go ahead and create a pull request,
|
* If you have a trivial fix or improvement, go ahead and create a pull request,
|
||||||
addressing (with `@...`) a suitable maintainer of this repository (see
|
addressing (with `@...`) a suitable maintainer of this repository (see
|
||||||
[MAINTAINERS.md](MAINTAINERS.md)) in the description of the pull request.
|
[MAINTAINERS.md](MAINTAINERS.md)) in the description of the pull request.
|
||||||
|
@ -16,3 +18,34 @@ Prometheus uses GitHub to manage reviews of pull requests.
|
||||||
and the _Formatting and style_ section of Peter Bourgon's [Go: Best
|
and the _Formatting and style_ section of Peter Bourgon's [Go: Best
|
||||||
Practices for Production
|
Practices for Production
|
||||||
Environments](http://peter.bourgon.org/go-in-production/#formatting-and-style).
|
Environments](http://peter.bourgon.org/go-in-production/#formatting-and-style).
|
||||||
|
|
||||||
|
|
||||||
|
## Steps to Contribute
|
||||||
|
|
||||||
|
Should you wish to work on an issue, please claim it first by commenting on the GitHub issue that you want to work on it. This is to prevent duplicated efforts from contributors on the same issue.
|
||||||
|
|
||||||
|
Please check the [`low-hanging-fruit`](https://github.com/prometheus/prometheus/issues?q=is%3Aissue+is%3Aopen+label%3A%22low+hanging+fruit%22) label to find issues that are good for getting started. If you have questions about one of the issues, with or without the tag, please comment on them and one of the maintainers will clarify it. For a quicker response, contact us over [IRC](https://prometheus.io/community).
|
||||||
|
|
||||||
|
For complete instructions on how to compile see: [Building From Source](https://github.com/prometheus/prometheus#building-from-source)
|
||||||
|
|
||||||
|
For quickly compiling and testing your changes do:
|
||||||
|
```
|
||||||
|
# For building.
|
||||||
|
go build ./cmd/prometheus/
|
||||||
|
./prometheus
|
||||||
|
|
||||||
|
# For testing.
|
||||||
|
make test # Make sure all the tests pass before you commit and push :)
|
||||||
|
```
|
||||||
|
|
||||||
|
All our issues are regularly tagged so that you can also filter down the issues involving the components you want to work on. For our labelling policy refer [the wiki page](https://github.com/prometheus/prometheus/wiki/Label-Names-and-Descriptions).
|
||||||
|
|
||||||
|
## Pull Request Checklist
|
||||||
|
|
||||||
|
* Branch from the master branch and, if needed, rebase to the current master branch before submitting your pull request. If it doesn't merge cleanly with master you may be asked to rebase your changes.
|
||||||
|
|
||||||
|
* Commits should be as small as possible, while ensuring that each commit is correct independently (i.e., each commit should compile and pass tests).
|
||||||
|
|
||||||
|
* If your patch is not getting reviewed or you need a specific person to review it, you can @-reply a reviewer asking for a review in the pull request or a comment, or you can ask for a review on IRC channel [#prometheus](https://webchat.freenode.net/?channels=#prometheus) on irc.freenode.net (for the easiest start, [join via Riot](https://riot.im/app/#/room/#prometheus:matrix.org)).
|
||||||
|
|
||||||
|
* Add tests relevant to the fixed bug or new feature.
|
||||||
|
|
Loading…
Reference in a new issue