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

Remove the capture_disjoint_fields feature #108550

Merged
merged 1 commit into from
Mar 1, 2023

Conversation

clubby789
Copy link
Contributor

As best I can tell, this was stabilized for Edition 2021 in #88126 but the feature was never removed.

@rustbot
Copy link
Collaborator

rustbot commented Feb 28, 2023

r? @Nilstrieb

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

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 28, 2023
@compiler-errors
Copy link
Member

I don't think we ever plan on enabling this for editions < 2021, and only nightly(or bootstrap) users are able to use it anyways. Therefore I think this is totally fine to remove.

I don't think this needs an FCP or anything, since the feature was already stabilized for edition 2021+, but cc @rust-lang/lang for good measure. This is a small PR and can be reverted just in case I'm totally misunderstanding the process here.

r? @compiler-errors @bors r+

@bors
Copy link
Contributor

bors commented Feb 28, 2023

📌 Commit f83ce99 has been approved by compiler-errors

It is now in the queue for this repository.

@rustbot rustbot assigned compiler-errors and unassigned Noratrieb Feb 28, 2023
@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 Feb 28, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 28, 2023
…ler-errors

Remove the `capture_disjoint_fields` feature

As best I can tell, this was stabilized for Edition 2021 in rust-lang#88126 but the feature was never removed.
@scottmcm
Copy link
Member

This makes perfect sense to me, @compiler-errors 👍

And in general I'd say compiler's always allowed to remove nightly-only stuff it deems not useful enough to keep maintaining.

Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Feb 28, 2023
…ler-errors

Remove the `capture_disjoint_fields` feature

As best I can tell, this was stabilized for Edition 2021 in rust-lang#88126 but the feature was never removed.
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 28, 2023
…ler-errors

Remove the `capture_disjoint_fields` feature

As best I can tell, this was stabilized for Edition 2021 in rust-lang#88126 but the feature was never removed.
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 1, 2023
…iaskrgr

Rollup of 8 pull requests

Successful merges:

 - rust-lang#108297 (Exit when there are unmatched delims to avoid noisy diagnostics)
 - rust-lang#108531 (rustdoc: Show that repeated expression arrays can be made with constant values)
 - rust-lang#108536 (Update books)
 - rust-lang#108550 (Remove the `capture_disjoint_fields` feature)
 - rust-lang#108551 (Descriptive error when users try to combine RPITIT/AFIT with specialization)
 - rust-lang#108554 (Only look for param in item's generics if it actually comes from generics)
 - rust-lang#108555 (Fix a race in the query system)
 - rust-lang#108558 (add missing feature in core/tests)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 78f9bb1 into rust-lang:master Mar 1, 2023
@rustbot rustbot added this to the 1.69.0 milestone Mar 1, 2023
@jackh726 jackh726 mentioned this pull request Mar 1, 2023
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants