Skip to content

Allow rust-highfive to label issues it creates. #72395

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
May 23, 2020
Merged

Allow rust-highfive to label issues it creates. #72395

merged 1 commit into from
May 23, 2020

Conversation

Elinvynia
Copy link
Contributor

This is my first meaningful PR, I am unsure how to test this code so any pointers would be welcome!

I am about 50% sure it works.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 20, 2020
@Elinvynia
Copy link
Contributor Author

cc @RalfJung

@RalfJung
Copy link
Member

Cc @kennytm @pietroalbini

@nikomatsakis
Copy link
Contributor

r? @Mark-Simulacrum (feel free to further re-assign...)

Copy link
Member

@kennytm kennytm left a comment

Choose a reason for hiding this comment

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

the code looks fine to me, except that LABELS could use a list directly rather than sets.

cc @rust-lang/wg-triage, are you ok with these choices of labels?

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

Works for me!

@Mark-Simulacrum
Copy link
Member

@Elinvynia Could you update the sets to lists? With that done I think we can merge this in. Thanks!

@Mark-Simulacrum
Copy link
Member

Can you also squash the commits into one? Thanks!

r=me with that done

@Elinvynia
Copy link
Contributor Author

?r=me

@RalfJung
Copy link
Member

"r=me" is code for "tell bors to approve this PR in my name". It is also entire inscrutable unless you already know it, and contributors without bors permission cannot use it anyway.

But it tells me I can now do this:
@bors r=Mark-Simulacrum

@bors
Copy link
Collaborator

bors commented May 22, 2020

📌 Commit dc4b9fd has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 22, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request May 23, 2020
Rollup of 7 pull requests

Successful merges:

 - rust-lang#71289 (Allow using `Self::` in doc)
 - rust-lang#72375 (Improve E0599 explanation)
 - rust-lang#72385 (Add some teams to prioritization exclude_labels)
 - rust-lang#72395 (Allow rust-highfive to label issues it creates.)
 - rust-lang#72453 (Add flag to open docs:  x.py doc --open)
 - rust-lang#72459 (Add core::future::IntoFuture)
 - rust-lang#72461 (Clean up E0600 explanation)

Failed merges:

r? @ghost
@bors bors merged commit 01adfe1 into rust-lang:master May 23, 2020
@Elinvynia Elinvynia deleted the highfive branch July 6, 2020 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants