Skip to content

fill match arms#733

Merged
bors[bot] merged 1 commit intorust-lang:masterfrom
gfreezy:fill-match-arm-assit
Feb 3, 2019
Merged

fill match arms#733
bors[bot] merged 1 commit intorust-lang:masterfrom
gfreezy:fill-match-arm-assit

Conversation

@gfreezy
Copy link
Copy Markdown
Contributor

@gfreezy gfreezy commented Feb 3, 2019

fixed #626

@gfreezy gfreezy force-pushed the fill-match-arm-assit branch from d75f3ca to bfaefed Compare February 3, 2019 16:28
@matklad
Copy link
Copy Markdown
Contributor

matklad commented Feb 3, 2019

bors r+

LGTM!

bors bot added a commit that referenced this pull request Feb 3, 2019
733: fill match arms r=matklad a=gfreezy

fixed #626 

Co-authored-by: gfreezy <gfreezy@gmail.com>
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Feb 3, 2019

Build succeeded

@bors bors bot merged commit bfaefed into rust-lang:master Feb 3, 2019
@gfreezy gfreezy deleted the fill-match-arm-assit branch February 4, 2019 02:20
Wilfred added a commit to Wilfred/rust-analyzer that referenced this pull request Mar 31, 2026
"Fill match arms" is an extremely helpful assist, but the name is
pretty opaque to new users (at least it was to me). We actually
renamed the file in rust-lang#10299 from
`fill_match_arms.rs` to `add_missing_match_arms.rs` but the label
hasn't changed from its original text in rust-lang#733.

Instead, show "Add missing match arms" if there are multiple missing
match arms, and show "Add missing match arm `Foo::Bar`" when there's
only a single missing match arm.

Partially generated by Claude Opus.
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.

Fill match arm assit

2 participants