@@ -35,7 +35,7 @@ There are a number of requirements that need to be met in order for reviewing to
35
35
36
36
## Reviewing Checklist
37
37
38
- The following list of questions (to be posted as part highfive bot 's automatic PR response)
38
+ The following list of questions (to be posted as part rustbot 's automatic PR response)
39
39
will help reviewers and PR authors alike to bring PRs into good shape and meet the above criteria:
40
40
41
41
> #### Checklist for PR authors and reviewers
@@ -77,15 +77,15 @@ However, sometimes there are gray areas where it is not immediately clear how to
77
77
This section lists a few common cases together with guidance on how to deal with them.
78
78
79
79
* ### I don't think I am a good fit for reviewing - what now?
80
- It is completely normal that you get assigned a PR (via highfive or otherwise) but don't feel comfortable reviewing it.
80
+ It is completely normal that you get assigned a PR (via rustbot or otherwise) but don't feel comfortable reviewing it.
81
81
Here is what you can do, depending on the concrete case:
82
82
83
83
- If the change seems really big or contentious, consider asking for an MCP (see below).
84
84
- If you know just the right person for the review, assign them via ` r? @<github-name> ` .
85
85
It's polite to leave a comment asking them if they can take over --
86
86
but you don't have to make sure beforehand that they can actually do it.
87
87
- If you don't know the code well or already have too much on your plate,
88
- ask highfive to roll the dice again via ` r? rust-lang/ compiler-contributors ` .
88
+ ask rustbot to roll the dice again via ` r? compiler ` .
89
89
90
90
You can also always ask for help on the ` #t-compiler/reviews ` Zulip stream for finding a reviewer.
91
91
That being said, you are always welcome to do an initial review (to the extent you are comformtable with)
0 commit comments