Skip to content
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

Add triagebot mentions. #98581

Merged
merged 1 commit into from
Jun 28, 2022
Merged

Add triagebot mentions. #98581

merged 1 commit into from
Jun 28, 2022

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Jun 27, 2022

This migrates the configuration of mentions from highfive to triagebot.

I also fixed a few broken paths (error_codes.rs src/librustdoc/html/static/themes src/librustdoc/html/static/themes/ayu.css).

@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(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 Jun 27, 2022
Copy link
Member

@Mark-Simulacrum Mark-Simulacrum left a comment

Choose a reason for hiding this comment

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

r=me, modulo nit if you want to resolve that so we don't need to do a rename dance

triagebot.toml Outdated
Changes rustc_apfloat. rustc_apfloat is currently in limbo and you almost
certainly don't want to change it (see #55993).
"""
reviewers = ["@eddyb"]
Copy link
Member

Choose a reason for hiding this comment

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

I'm wondering if we should make clearer (i.e., adjust triagebot to have the field name be "cc") that these are really just cc (not really reviewers) -- we're not adding them to github or bors as required reviewers in any sense.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yea, that sounds good. I was just copying the highfive layout. I opened rust-lang/triagebot#1628 to rename it.

@ehuss ehuss force-pushed the triagebot-mentions branch from 368e275 to 44e5715 Compare June 27, 2022 18:34
@ehuss
Copy link
Contributor Author

ehuss commented Jun 27, 2022

Updated to rename to cc.

@bors r=Mark-Simulacrum rollup

@bors
Copy link
Contributor

bors commented Jun 27, 2022

📌 Commit 44e5715 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 Jun 27, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 27, 2022
…askrgr

Rollup of 9 pull requests

Successful merges:

 - rust-lang#98331 (Fix rustdoc argument error)
 - rust-lang#98506 (Fix span issues in object safety suggestions)
 - rust-lang#98563 (interpret: refactor allocation info query)
 - rust-lang#98576 (small regions refactoring)
 - rust-lang#98577 (Fix "kind" for associated types in trait implementations in rustdoc JSON)
 - rust-lang#98578 (Remove eddyb from miri failure pings)
 - rust-lang#98579 (liballoc tests: avoid int2ptr cast)
 - rust-lang#98581 (Add triagebot mentions.)
 - rust-lang#98587 (libcore tests: avoid int2ptr casts)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 19a05b5 into rust-lang:master Jun 28, 2022
@rustbot rustbot added this to the 1.64.0 milestone Jun 28, 2022

[mentions."compiler/rustc_mir_transform/src/"]
message = "Some changes occurred to MIR optimizations"
cc = ["@rust-lang/mir-opt"]
Copy link
Member

Choose a reason for hiding this comment

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

This should have been "wg-mir-opt".

I have not audited the other mention configs, except for the miri ones which seem fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

#98619 is open to fix it.

This comment was marked as duplicate.

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.

6 participants