Skip to content

Fix "Implement default members" to resolve IdentPat #20432

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

Conversation

sgasho
Copy link
Contributor

@sgasho sgasho commented Aug 11, 2025

closes: #20215

Evidence

Before "Implement default members"

スクリーンショット 2025-08-11 22 21 54

After

e.g. in "is_empty" method

Before this PR(: (Unbounded, ) | (, Unbounded) => true,
this PR: (std::ops::Bound::Unbounded, ) | (, std::ops::Bound::Unbounded) => true,

スクリーンショット 2025-08-11 22 22 19

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 11, 2025
Copy link
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, just some minor changes.

sgasho added a commit to sgasho/rust-analyzer that referenced this pull request Aug 12, 2025
@rustbot

This comment has been minimized.

@sgasho sgasho force-pushed the fix/20215_implement_default_member_to_resolve_ident_pat branch from 7910a8c to 6beab21 Compare August 12, 2025 12:48
@ChayimFriedman2
Copy link
Contributor

Can you squash please?

@sgasho sgasho force-pushed the fix/20215_implement_default_member_to_resolve_ident_pat branch from 6beab21 to 8ab6837 Compare August 12, 2025 12:54
@sgasho
Copy link
Contributor Author

sgasho commented Aug 12, 2025

@ChayimFriedman2
All commits were squashed and all tests are passed!
Thank you for reviewing!

@sgasho sgasho requested a review from ChayimFriedman2 August 12, 2025 13:15
Copy link
Contributor

@ChayimFriedman2 ChayimFriedman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Aug 12, 2025
Merged via the queue into rust-lang:master with commit b2a58b8 Aug 12, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 12, 2025
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.

"Implement default member" assist on trait messes up on pattern
3 participants