Skip to content

fix empty_structs_with_brackets suggestion errors#10141

Merged
bors merged 1 commit into
rust-lang:masterfrom
FoseFx:fosefx/9887
Jan 3, 2023
Merged

fix empty_structs_with_brackets suggestion errors#10141
bors merged 1 commit into
rust-lang:masterfrom
FoseFx:fosefx/9887

Conversation

@FoseFx

@FoseFx FoseFx commented Jan 1, 2023

Copy link
Copy Markdown
Contributor

fixes #9887

I refer to my comment to explain this PR.


changelog: Sugg: [empty_structs_with_brackets]: The suggestion is no longer machine applicable, to avoid errors when accessing struct fields
#10141

@rustbot

rustbot commented Jan 1, 2023

Copy link
Copy Markdown
Collaborator

r? @Jarcho

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 1, 2023
Comment thread tests/ui/empty_structs_with_brackets.rs
@Jarcho

Jarcho commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

Thank you. @bors r+

@bors

bors commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

📌 Commit 7a1a9c8 has been approved by Jarcho

It is now in the queue for this repository.

@bors

bors commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

⌛ Testing commit 7a1a9c8 with merge 8b1ac45...

@bors

bors commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Jarcho
Pushing 8b1ac45 to master...

@bors bors merged commit 8b1ac45 into rust-lang:master Jan 3, 2023
@FoseFx FoseFx deleted the fosefx/9887 branch January 3, 2023 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FP empty_structs_with_brackets

4 participants