-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Remove libs message about ACPs from triagebot #111679
Conversation
(rustbot has picked a reviewer for you, use r? to override) |
cc @rust-lang/libs-api |
The libs team is currently reworking the ACP process and we don't want to encourage people to submit new ACPs in the meantime.
3e25913
to
67c0cfe
Compare
If this PR contains changes to any `rust-lang/rust` public library APIs then \ | ||
please comment with `@rustbot label +T-libs-api -T-libs` to tag it \ | ||
appropriately. If this PR contains changes to any unstable APIs please edit \ | ||
the PR description to add a link to the relevant [API Change \ | ||
Proposal](https://std-dev-guide.rust-lang.org/feature-lifecycle/api-change-proposals.html) \ | ||
or [create one](https://github.com/rust-lang/libs-team/issues/new?assignees=&labels=api-change-proposal%2C+T-libs-api&template=api-change-proposal.md&title=%28My+API+Change+Proposal%29) \ | ||
if you haven't already. If you're unsure where your change falls no worries, \ | ||
just leave it as is and the reviewer will take a look and make a decision to \ | ||
forward on if necessary. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it still be nice to tell people to appropriately add labels? (would it be fine to only remove the sentence on ACPs?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll be revising how we use the labels and categorize PRs, so this will be updated as well. For now this PR is just removing outdated information while we figure out what the right process/info should be.
@bors r+ rollup |
Rollup of 7 pull requests Successful merges: - rust-lang#110884 (Support RISC-V unaligned-scalar-mem target feature) - rust-lang#111160 (Update serde in workspace and non-synced dependencies) - rust-lang#111168 (Specialize ToString implementation for fmt::Arguments) - rust-lang#111527 (add examples of port 0 binding behavior) - rust-lang#111561 (Include better context for "already exists" error in compiletest) - rust-lang#111633 (Avoid `&format("...")` calls in error message code.) - rust-lang#111679 (Remove libs message about ACPs from triagebot) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
The libs team is currently reworking the ACP process and we don't want to encourage people to submit new ACPs in the meantime.