Skip to content

Deprecate the from_iter_instead_of_collect lint - #17208

Merged
llogiq merged 1 commit into
rust-lang:masterfrom
samueltardieu:lints/remove-from-iter-instead-of-collect
Jun 11, 2026
Merged

llogiq merged 1 commit into
rust-lang:masterfrom
samueltardieu:lints/remove-from-iter-instead-of-collect

Conversation

@samueltardieu

Copy link
Copy Markdown
Member

As discussed in
https://rust-lang.zulipchat.com/#narrow/channel/559984-clippy.2Fmeetings/topic/Meeting.202026-06-09/with/601758524

changelog: [from_iter_instead_of_collect]: deprecate lint which has proved to be problematic for very little gain

@rustbot rustbot added needs-fcp PRs that add, remove, or rename lints and need an FCP S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Jun 11, 2026
@rustbot

rustbot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

r? @llogiq

rustbot has assigned @llogiq.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: 8 candidates
  • 8 candidates expanded to 8 candidates
  • Random selection from Jarcho, dswij, llogiq

@llogiq
llogiq enabled auto-merge June 11, 2026 10:54
@llogiq
llogiq added this pull request to the merge queue Jun 11, 2026
@github-actions

Copy link
Copy Markdown

Lintcheck changes for ab9ee11

Lint Added Removed Changed
clippy::from_iter_instead_of_collect 0 11 0

This comment will be updated if you push new changes

Merged via the queue into rust-lang:master with commit 917d94a Jun 11, 2026
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 11, 2026
@samueltardieu
samueltardieu deleted the lints/remove-from-iter-instead-of-collect branch June 11, 2026 11:30
epage added a commit to epage/_rust that referenced this pull request Aug 9, 2026
`clippy::from_iter_instead_of_collect` was
[deprecated](rust-lang/rust-clippy#17208) a
couple of months ago, and shows a warning on `beta` (`1.98.0`) when
there is any config set for it:
```
warning: lint `clippy::from_iter_instead_of_collect` has been removed: lint has proved problematic
  |
  = note: requested on the command line with `-W clippy::from_iter_instead_of_collect`
  = note: `#[warn(renamed_and_removed_lints)]` on by default
```

Given that `1.98.0` will be `stable` in a couple of weeks
(`2026-08-20`), I figured it would be a good idea to remove
`from_iter_instead_of_collect` from `[lints.clippy]` early.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-fcp PRs that add, remove, or rename lints and need an FCP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants