Skip to content

Add reference PRs for r? and r+ comments #928

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

Merged
merged 1 commit into from
Oct 21, 2020

Conversation

JohnTitor
Copy link
Member

I think this would also help new contributors understand how the comment works.

@JohnTitor
Copy link
Member Author

I'd like to add some more changes, one sec.

@JohnTitor JohnTitor changed the title Add a reference PR for r? [at]someone Add reference PRs for r? and r+ comments Oct 21, 2020
@JohnTitor
Copy link
Member Author

Add a reference for r+ as well, it's ready to review :)

@jyn514 jyn514 added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Oct 21, 2020
Copy link
Member

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

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

LGTM but you need to rebase over your other PR ;)

@jyn514 jyn514 added S-waiting-on-author Status: this PR is waiting for additional action by the OP and removed S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content labels Oct 21, 2020
@JohnTitor
Copy link
Member Author

Rebased!

@LeSeulArtichaut LeSeulArtichaut merged commit 47893ba into rust-lang:master Oct 21, 2020
@JohnTitor JohnTitor deleted the example-comment branch October 21, 2020 21:54
jyn514 added a commit to jyn514/rustc-dev-guide that referenced this pull request Oct 21, 2020
Example usage:

```
$ ./check-in.sh
usage: ./check-in.sh <since> <number-of-prs-merged>
$ ./check-in.sh 2020-09-03 72
Authors:
- **@1c3t3a**
- **@arora-aman**
... snip ...
Changes:
- Replace links to `buildbot2.r-l.o` with `bors.r-l.o` [rust-lang#929](rust-lang#929)
- Add reference PRs for `r?` and `r+` comments [rust-lang#928](rust-lang#928)
... snip ...
Changes in progress:
```
jyn514 added a commit to jyn514/rustc-dev-guide that referenced this pull request Oct 21, 2020
Example usage:

```
$ ./check-in.sh
usage: ./check-in.sh <since> <number-of-prs-merged>
$ ./check-in.sh 2020-09-03 72
Authors:
- **@1c3t3a**
- **@arora-aman**
... snip ...
Changes:
- Replace links to `buildbot2.r-l.o` with `bors.r-l.o` [rust-lang#929](rust-lang#929)
- Add reference PRs for `r?` and `r+` comments [rust-lang#928](rust-lang#928)
... snip ...
Changes in progress:
```
jyn514 added a commit to jyn514/rustc-dev-guide that referenced this pull request Oct 21, 2020
Example usage:

```
$ ./check-in.sh
usage: ./check-in.sh <since> <number-of-prs-merged>
$ ./check-in.sh 2020-09-03
usage: ./check-in.sh <since> <number-of-prs-merged>
help: you can find the number of PRs merged at https://github.com/rust-lang/rustc-dev-guide/pulls?q=is%3Apr+is%3Aclosed+updated%3A%3E2020-09-03
$ ./check-in.sh 2020-09-03 72
Authors:
- **@1c3t3a**
- **@arora-aman**
... snip ...
Changes:
- Replace links to `buildbot2.r-l.o` with `bors.r-l.o` [rust-lang#929](rust-lang#929)
- Add reference PRs for `r?` and `r+` comments [rust-lang#928](rust-lang#928)
... snip ...
Changes in progress:
```
jyn514 added a commit that referenced this pull request Oct 22, 2020
Example usage:

```
$ ./check-in.sh
usage: ./check-in.sh <since> <number-of-prs-merged>
$ ./check-in.sh 2020-09-03
usage: ./check-in.sh <since> <number-of-prs-merged>
help: you can find the number of PRs merged at https://github.com/rust-lang/rustc-dev-guide/pulls?q=is%3Apr+is%3Aclosed+updated%3A%3E2020-09-03
$ ./check-in.sh 2020-09-03 72
Authors:
- **@1c3t3a**
- **@arora-aman**
... snip ...
Changes:
- Replace links to `buildbot2.r-l.o` with `bors.r-l.o` [#929](#929)
- Add reference PRs for `r?` and `r+` comments [#928](#928)
... snip ...
Changes in progress:
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: this PR is waiting for additional action by the OP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants