Exclude relnotes-tracking-issue from needs-triage by Noratrieb · Pull Request #132825 · rust-lang/rust (original) (raw)
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 andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation4 Commits1 Checks6 Files changed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.
Use r? to explicitly pick a reviewer
Area: Issues & PRs about the rust-lang/rust repository itself
Status: Awaiting review from the assignee but also interested parties.
labels
triagebot.toml has been modified, there may have been changes to the review queue.
📌 Commit 2b46960 has been approved by clubby789
It is now in the queue for this repository.
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
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request
…bby789
Exclude relnotes-tracking-issue from needs-triage
bors added a commit to rust-lang-ci/rust that referenced this pull request
…iaskrgr
Rollup of 4 pull requests
Successful merges:
- rust-lang#131258 (Stabilize s390x inline assembly)
- rust-lang#132577 (Report the
unexpected_cfgslint in external macros) - rust-lang#132801 (interpret: get_alloc_info: also return mutability)
- rust-lang#132825 (Exclude relnotes-tracking-issue from needs-triage)
r? @ghost
@rustbot modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request
…kingjubilee
Rollup of 6 pull requests
Successful merges:
- rust-lang#131258 (Stabilize s390x inline assembly)
- rust-lang#132801 (interpret: get_alloc_info: also return mutability)
- rust-lang#132823 (require const_impl_trait gate for all conditional and trait const calls)
- rust-lang#132824 (Update grammar in wasm-c-abi's compiler flag documentation)
- rust-lang#132825 (Exclude relnotes-tracking-issue from needs-triage)
- rust-lang#132828 (Additional tests to ensure let is rejected during parsing)
r? @ghost
@rustbot modify labels: rollup
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request
Rollup merge of rust-lang#132825 - Noratrieb:Noratrieb-patch-1, r=clubby789
Exclude relnotes-tracking-issue from needs-triage
mati865 pushed a commit to mati865/rust that referenced this pull request
…bby789
Exclude relnotes-tracking-issue from needs-triage
mati865 pushed a commit to mati865/rust that referenced this pull request
…kingjubilee
Rollup of 6 pull requests
Successful merges:
- rust-lang#131258 (Stabilize s390x inline assembly)
- rust-lang#132801 (interpret: get_alloc_info: also return mutability)
- rust-lang#132823 (require const_impl_trait gate for all conditional and trait const calls)
- rust-lang#132824 (Update grammar in wasm-c-abi's compiler flag documentation)
- rust-lang#132825 (Exclude relnotes-tracking-issue from needs-triage)
- rust-lang#132828 (Additional tests to ensure let is rejected during parsing)
r? @ghost
@rustbot modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request
Don't exclude relnotes from needs-triage label
So initially we didn't exclude needs-triage label, then we did exclude them in rust-lang#132825 as sometimes the needs-triage is redundant. However, I think they are probably worth double-checking because often some of the labels are only accurate/relevant for the implementation PR, but not for the purposes of the relnotes tracking issue. Furthermore, sometimes relevant team labels can be removed. So to make it less likely for relnotes to slip through, I think we should still label relnotes-tracking-issues with needs-triage.
cc https://rust-lang.zulipchat.com/#narrow/channel/241545-t-release/topic/Please.20CC.20lang
r? release
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request
Don't exclude relnotes from needs-triage label
So initially we didn't exclude needs-triage label, then we did exclude them in rust-lang#132825 as sometimes the needs-triage is redundant. However, I think they are probably worth double-checking because often some of the labels are only accurate/relevant for the implementation PR, but not for the purposes of the relnotes tracking issue. Furthermore, sometimes relevant team labels can be removed. So to make it less likely for relnotes to slip through, I think we should still label relnotes-tracking-issues with needs-triage.
cc https://rust-lang.zulipchat.com/#narrow/channel/241545-t-release/topic/Please.20CC.20lang
r? release
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request
Rollup merge of rust-lang#133283 - jieyouxu:triage-relnotes, r=BoxyUwU
Don't exclude relnotes from needs-triage label
So initially we didn't exclude needs-triage label, then we did exclude them in rust-lang#132825 as sometimes the needs-triage is redundant. However, I think they are probably worth double-checking because often some of the labels are only accurate/relevant for the implementation PR, but not for the purposes of the relnotes tracking issue. Furthermore, sometimes relevant team labels can be removed. So to make it less likely for relnotes to slip through, I think we should still label relnotes-tracking-issues with needs-triage.
cc https://rust-lang.zulipchat.com/#narrow/channel/241545-t-release/topic/Please.20CC.20lang
r? release
Labels
Area: Issues & PRs about the rust-lang/rust repository itself
Status: Waiting on bors to run and complete tests. Bors will change the label on completion.