Skip to content

fix(config): improve error handling in Cfg::list_toolchains() - #5068

Merged
rami3l merged 3 commits into
rust-lang:mainfrom
rami3l:fix/error-handling-toolchain-list
Sep 12, 2026
Merged

fix(config): improve error handling in Cfg::list_toolchains()#5068
rami3l merged 3 commits into
rust-lang:mainfrom
rami3l:fix/error-handling-toolchain-list

Conversation

@rami3l

@rami3l rami3l commented Sep 11, 2026

Copy link
Copy Markdown
Member

This is another problem that I have noticed when working on rust-lang/goals#736.

Comment thread src/config.rs Outdated
@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch 4 times, most recently from 7fb9f17 to 3457ff5 Compare September 11, 2026 13:31
@rami3l
rami3l requested a review from djc September 11, 2026 13:40
@rami3l
rami3l marked this pull request as ready for review September 11, 2026 13:40
@rami3l
rami3l marked this pull request as draft September 11, 2026 13:47
@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch from 3457ff5 to 784946f Compare September 11, 2026 13:49
@rami3l
rami3l marked this pull request as ready for review September 11, 2026 13:50
Comment thread src/config.rs Outdated
Comment thread src/config.rs Outdated
@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch from 784946f to a755461 Compare September 11, 2026 14:43
@rami3l
rami3l requested a review from djc September 11, 2026 14:45
@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch 3 times, most recently from 4f4dc54 to e09a4c9 Compare September 12, 2026 07:41
@rustbot

This comment has been minimized.

Comment thread src/config.rs Outdated
@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch from e09a4c9 to 71671c4 Compare September 12, 2026 11:26
@rustbot

rustbot commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@rami3l
rami3l force-pushed the fix/error-handling-toolchain-list branch from 71671c4 to 0001952 Compare September 12, 2026 11:29
@rami3l
rami3l enabled auto-merge September 12, 2026 11:35
@rami3l
rami3l added this pull request to the merge queue Sep 12, 2026
Merged via the queue into rust-lang:main with commit 454ff04 Sep 12, 2026
32 checks passed
@rami3l
rami3l deleted the fix/error-handling-toolchain-list branch September 12, 2026 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants