blob: a2ac20deb2e5fadc8cbc608acbc0fcf8fef81e1d [file] [log] [blame] [view]
Steve Klabnikf645cad2015-02-13 17:26:441# Contributing to Rust
Tim Chevalier5afd7602013-10-21 19:10:572
Steve Klabnikf645cad2015-02-13 17:26:443Thank you for your interest in contributing to Rust! There are many ways to
4contribute, and we appreciate all of them. This document is a bit long, so here's
5links to the major sections:
Tim Chevalier5afd7602013-10-21 19:10:576
Steve Klabnikf645cad2015-02-13 17:26:447* [Feature Requests](#feature-requests)
8* [Bug Reports](#bug-reports)
9* [Pull Requests](#pull-requests)
10* [Writing Documentation](#writing-documentation)
11* [Issue Triage](#issue-triage)
12* [Out-of-tree Contributions](#out-of-tree-contributions)
Tim Chevalier5afd7602013-10-21 19:10:5713
Steve Klabnikf645cad2015-02-13 17:26:4414If you have questions, please make a post on [internals.rust-lang.org][internals] or
15hop on [#rust-internals][pound-rust-internals].
Nick Cameron852cef82014-09-22 00:46:2416
Darrell Hamiltona6b47c02015-02-19 03:53:0017As a reminder, all contributors are expected to follow our [Code of Conduct][coc].
Brian Anderson36c63a32015-01-02 23:41:4718
19[pound-rust-internals]: http://chat.mibbit.com/?server=irc.mozilla.org&channel=%23rust-internals
Brian Anderson94404072015-01-27 23:44:0520[internals]: http://internals.rust-lang.org
Steve Klabnikf645cad2015-02-13 17:26:4421[coc]: http://www.rust-lang.org/conduct.html
Brian Anderson36c63a32015-01-02 23:41:4722
Steve Klabnikf645cad2015-02-13 17:26:4423## Feature Requests
24
25To request a change to the way that the Rust language works, please open an
26issue in the [RFCs repository](https://github.com/rust-lang/rfcs/issues/new)
27rather than this one. New features and other significant language changes
28must go through the RFC process.
29
30## Bug Reports
31
32While bugs are unfortunate, they're a reality in software. We can't fix what we
33don't know about, so please report liberally. If you're not sure if something
34is a bug or not, feel free to file a bug anyway.
35
36If you have the chance, before reporting a bug, please [search existing
37issues](https://github.com/rust-lang/rust/search?q=&type=Issues&utf8=%E2%9C%93),
38as it's possible that someone else has already reported your error. This doesn't
39always work, and sometimes it's hard to know what to search for, so consider this
40extra credit. We won't mind if you accidentally file a duplicate report.
41
42Opening an issue is as easy as following [this
43link](https://github.com/rust-lang/rust/issues/new) and filling out the fields.
44Here's a template that you can use to file a bug, though it's not necessary to
45use it exactly:
46
47 <short summary of the bug>
48
49 I tried this code:
50
51 <code sample that causes the bug>
52
53 I expected to see this happen: <explanation>
54
55 Instead, this happened: <explanation>
56
57 ## Meta
58
59 `rustc --version --verbose`:
60
61 Backtrace:
62
63All three components are important: what you did, what you expected, what
64happened instead. Please include the output of `rustc --version --verbose`,
65which includes important information about what platform you're on, what
66version of Rust you're using, etc.
67
68Sometimes, a backtrace is helpful, and so including that is nice. To get
69a backtrace, set the `RUST_BACKTRACE` environment variable. The easiest way
70to do this is to invoke `rustc` like this:
71
72```bash
73$ RUST_BACKTRACE=1 rustc ...
74```
75
76## Pull Requests
77
78Pull requests are the primary mechanism we use to change Rust. GitHub itself
79has some [great documentation][pull-requests] on using the Pull Request
80feature. We use the 'fork and pull' model described there.
81
82[pull-requests]: https://help.github.com/articles/using-pull-requests/
83
84Please make pull requests against the `master` branch.
85
Steve Klabnik720da312015-07-06 18:46:2186Compiling all of `make check` can take a while. When testing your pull request,
87consider using one of the more specialized `make` targets to cut down on the
88amount of time you have to wait. You need to have built the compiler at least
89once before running these will work, but that’s only one full build rather than
90one each time.
91
92 $ make -j8 rustc-stage1 && make check-stage1
93
94is one such example, which builds just `rustc`, and then runs the tests. If
95you’re adding something to the standard library, try
96
97 $ make -j8 check-stage1-std NO_REBUILD=1
98
99This will not rebuild the compiler, but will run the tests.
100
Steve Klabnikf645cad2015-02-13 17:26:44101All pull requests are reviewed by another person. We have a bot,
edunham177531e2015-05-27 14:29:02102@rust-highfive, that will automatically assign a random person to review your
103request.
Steve Klabnikf645cad2015-02-13 17:26:44104
105If you want to request that a specific person reviews your pull request,
106you can add an `r?` to the message. For example, Steve usually reviews
107documentation changes. So if you were to make a documentation change, add
108
109 r? @steveklabnik
110
111to the end of the message, and @rust-highfive will assign @steveklabnik instead
112of a random person. This is entirely optional.
113
114After someone has reviewed your pull request, they will leave an annotation
115on the pull request with an `r+`. It will look something like this:
116
117 @bors: r+ 38fe8d2
118
119This tells @bors, our lovable integration bot, that your pull request has
120been approved. The PR then enters the [merge queue][merge-queue], where @bors
121will run all the tests on every platform we support. If it all works out,
122@bors will merge your code into `master` and close the pull request.
123
124[merge-queue]: http://buildbot.rust-lang.org/homu/queue/rust
125
126## Writing Documentation
127
128Documentation improvements are very welcome. The source of `doc.rust-lang.org`
129is located in `src/doc` in the tree, and standard API documentation is generated
130from the source code itself.
131
132Documentation pull requests function in the same as other pull requests, though
133you may see a slightly different form of `r+`:
134
135 @bors: r+ 38fe8d2 rollup
136
137That additional `rollup` tells @bors that this change is eligible for a 'rollup'.
138To save @bors some work, and to get small changes through more quickly, when
139@bors attempts to merge a commit that's rollup-eligible, it will also merge
140the other rollup-eligible patches too, and they'll get tested and merged at
141the same time.
142
edunham177531e2015-05-27 14:29:02143To find documentation-related issues, sort by the [A-docs label][adocs].
144
145[adocs]: https://github.com/rust-lang/rust/issues?q=is%3Aopen+is%3Aissue+label%3AA-docs
146
Steve Klabnikf645cad2015-02-13 17:26:44147## Issue Triage
148
149Sometimes, an issue will stay open, even though the bug has been fixed. And
150sometimes, the original bug may go stale because something has changed in the
151meantime.
152
153It can be helpful to go through older bug reports and make sure that they are
154still valid. Load up an older issue, double check that it's still true, and
edunham177531e2015-05-27 14:29:02155leave a comment letting us know if it is or is not. The [least recently
156updated sort][lru] is good for finding issues like this.
Steve Klabnikf645cad2015-02-13 17:26:44157
edunham177531e2015-05-27 14:29:02158Contributors with sufficient permissions on the Rust repo can help by adding
159labels to triage issues:
160
161* Yellow, **A**-prefixed labels state which **area** of the project an issue
162 relates to.
163
164* Magenta, **B**-prefixed labels identify bugs which **belong** elsewhere.
165
166* Green, **E**-prefixed labels explain the level of **experience** necessary
167 to fix the issue.
168
169* Red, **I**-prefixed labels indicate the **importance** of the issue. The
170 [I-nominated][inom] label indicates that an issue has been nominated for
171 prioritizing at the next triage meeting.
172
173* Orange, **P**-prefixed labels indicate a bug's **priority**. These labels
174 are only assigned during triage meetings, and replace the [I-nominated][inom]
175 label.
176
177* Blue, **T**-prefixed bugs denote which **team** the issue belongs to.
178
179* Dark blue, **beta-** labels track changes which need to be backported into
180 the beta branches.
181
182* The purple **metabug** label marks lists of bugs collected by other
183 categories.
184
185If you're looking for somewhere to start, check out the [E-easy][eeasy] tag.
186
187[inom]: https://github.com/rust-lang/rust/issues?q=is%3Aopen+is%3Aissue+label%3AI-nominated
188[eeasy]: https://github.com/rust-lang/rust/issues?q=is%3Aopen+is%3Aissue+label%3AE-easy
Steve Klabnikf645cad2015-02-13 17:26:44189[lru]: https://github.com/rust-lang/rust/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-asc
190
191## Out-of-tree Contributions
192
193There are a number of other ways to contribute to Rust that don't deal with
194this repository.
195
196Answer questions in [#rust][pound-rust], or on [users.rust-lang.org][users],
197or on [StackOverflow][so].
198
199Participate in the [RFC process](https://github.com/rust-lang/rfcs).
200
201Find a [requested community library][community-library], build it, and publish
202it to [Crates.io](http://crates.io). Easier said than done, but very, very
203valuable!
204
205[pound-rust]: http://chat.mibbit.com/?server=irc.mozilla.org&channel=%23rust
206[users]: http://users.rust-lang.org/
207[so]: http://stackoverflow.com/questions/tagged/rust
208[community-library]: https://github.com/rust-lang/rfcs/labels/A-community-library