Update t-compiler beta nomination Zulip msg#147527
Conversation
b69b451 to
c340aa0
Compare
|
|
|
rustbot has assigned @Mark-Simulacrum. Use |
e4c6a54 to
dcb6bce
Compare
|
maybe r? @Kobzol (as soon as GitHub starts working again) |
|
We can always change the wording later, the current wording is fine by me. |
|
r? compiler (for someone to approve and merge 🙂 ) |
There was a problem hiding this comment.
Sorry, forgot to send one comment. I was mostly waiting for other people to chime in, but if no one has a comment in a ~day, you can r=me.
@bors delegate+
triagebot.toml
Outdated
| @@ -759,6 +765,7 @@ zulip_stream = 474880 # #t-compiler/backports | |||
| topic = "#{number}: beta-nominated" | |||
| message_on_add = "PR #{number} has been **accepted** for **beta** backport." | |||
|
|
|||
| # This is disabled | |||
There was a problem hiding this comment.
I would just remove this if it doesn't do anything. We can always create it again in the future if we redesign how it works.
There was a problem hiding this comment.
Are those really disabled? I don't see anything in the triagebot changes that makes them disabled. So same as @Kobzol comment, let's actually remove them.
There was a problem hiding this comment.
Stable auto-nomination was disabled in #147263 so I thought it was confusing having these bits of configuration which do nothing.
But OK I will remove the comment.
dcb6bce to
de67ea3
Compare
|
@bors r=Kobzol rollup |
Rollup of 11 pull requests Successful merges: - #143191 (Stabilize `rwlock_downgrade` library feature) - #147444 (Allow printing a fully-qualified path in `def_path_str`) - #147527 (Update t-compiler beta nomination Zulip msg) - #147670 (some `ErrorGuaranteed` cleanups) - #147676 (Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes) - #147708 (const `mem::drop`) - #147710 (Fix ICE when using contracts on async functions) - #147716 (Fix some comments) - #147718 (miri: use allocator_shim_contents codegen helper) - #147729 (ignore boring locals when explaining why a borrow contains a point due to drop of a live local under polonius) - #147742 (Revert unintentional whitespace changes to rustfmt-excluded file) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of #147527 - apiraino:update-beta-nom-zulip-msg, r=Kobzol Update t-compiler beta nomination Zulip msg Sister patch of [triagebot#2191](rust-lang/triagebot#2191) Follow-up to #147263 The triagebot now triggers a different message when a PR is nominated for backport, making it look like more a suggestion to evaluate for the author/reviewers than a mandatory decision for the team to take. The wording (as per [triagebot#2191](rust-lang/triagebot#2191)) is open to suggestions. Thanks
Rollup of 11 pull requests Successful merges: - rust-lang/rust#143191 (Stabilize `rwlock_downgrade` library feature) - rust-lang/rust#147444 (Allow printing a fully-qualified path in `def_path_str`) - rust-lang/rust#147527 (Update t-compiler beta nomination Zulip msg) - rust-lang/rust#147670 (some `ErrorGuaranteed` cleanups) - rust-lang/rust#147676 (Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes) - rust-lang/rust#147708 (const `mem::drop`) - rust-lang/rust#147710 (Fix ICE when using contracts on async functions) - rust-lang/rust#147716 (Fix some comments) - rust-lang/rust#147718 (miri: use allocator_shim_contents codegen helper) - rust-lang/rust#147729 (ignore boring locals when explaining why a borrow contains a point due to drop of a live local under polonius) - rust-lang/rust#147742 (Revert unintentional whitespace changes to rustfmt-excluded file) r? `@ghost` `@rustbot` modify labels: rollup
Rollup of 11 pull requests Successful merges: - rust-lang/rust#143191 (Stabilize `rwlock_downgrade` library feature) - rust-lang/rust#147444 (Allow printing a fully-qualified path in `def_path_str`) - rust-lang/rust#147527 (Update t-compiler beta nomination Zulip msg) - rust-lang/rust#147670 (some `ErrorGuaranteed` cleanups) - rust-lang/rust#147676 (Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes) - rust-lang/rust#147708 (const `mem::drop`) - rust-lang/rust#147710 (Fix ICE when using contracts on async functions) - rust-lang/rust#147716 (Fix some comments) - rust-lang/rust#147718 (miri: use allocator_shim_contents codegen helper) - rust-lang/rust#147729 (ignore boring locals when explaining why a borrow contains a point due to drop of a live local under polonius) - rust-lang/rust#147742 (Revert unintentional whitespace changes to rustfmt-excluded file) r? `@ghost` `@rustbot` modify labels: rollup
…iaskrgr Rollup of 11 pull requests Successful merges: - rust-lang#143191 (Stabilize `rwlock_downgrade` library feature) - rust-lang#147444 (Allow printing a fully-qualified path in `def_path_str`) - rust-lang#147527 (Update t-compiler beta nomination Zulip msg) - rust-lang#147670 (some `ErrorGuaranteed` cleanups) - rust-lang#147676 (Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes) - rust-lang#147708 (const `mem::drop`) - rust-lang#147710 (Fix ICE when using contracts on async functions) - rust-lang#147716 (Fix some comments) - rust-lang#147718 (miri: use allocator_shim_contents codegen helper) - rust-lang#147729 (ignore boring locals when explaining why a borrow contains a point due to drop of a live local under polonius) - rust-lang#147742 (Revert unintentional whitespace changes to rustfmt-excluded file) r? `@ghost` `@rustbot` modify labels: rollup
Sister patch of triagebot#2191
Follow-up to #147263
The triagebot now triggers a different message when a PR is nominated for backport, making it look like more a suggestion to evaluate for the author/reviewers than a mandatory decision for the team to take.
The wording (as per triagebot#2191) is open to suggestions.
Thanks