Skip to content

Don't lint let_unit_value when () is explicit#10844

Merged
bors merged 1 commit into
rust-lang:masterfrom
Centri3:let_unit_value
Jan 5, 2024
Merged

Don't lint let_unit_value when () is explicit#10844
bors merged 1 commit into
rust-lang:masterfrom
Centri3:let_unit_value

Conversation

@Centri3

@Centri3 Centri3 commented May 30, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

since these are explicitly written (and not the result of a function call or anything else), they should be allowed, as they are both useful in some cases described in #9048

Fixes #9048

changelog: [let_unit_value]: Don't lint when () is explicit

@rustbot

rustbot commented May 30, 2023

Copy link
Copy Markdown
Collaborator

r? @llogiq

(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 May 30, 2023
llogiq
llogiq reviewed Jun 4, 2023
View reviewed changes

@llogiq llogiq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for taking this on, also I much like the additions to the tests. One thing that befuddled me is why did you regress from a let chain to an if_chain! macro invocation?

Comment thread clippy_lints/src/unit_types/let_unit_value.rs Outdated
@danielhenrymantilla danielhenrymantilla mentioned this pull request Jun 12, 2023
Merged
danielhenrymantilla
danielhenrymantilla reviewed Jun 12, 2023
View reviewed changes
Comment thread clippy_lints/src/unit_types/let_unit_value.rs Outdated
Comment thread tests/ui/let_unit.fixed Outdated
danielhenrymantilla
danielhenrymantilla reviewed Jun 12, 2023
View reviewed changes
Comment thread tests/ui/let_unit.stderr
@Centri3 Centri3 mentioned this pull request Aug 2, 2023
Open
@bors

bors commented Aug 11, 2023

Copy link
Copy Markdown
Contributor

☔ The latest upstream changes (presumably #11239) made this pull request unmergeable. Please resolve the merge conflicts.

@Alexendoo

Alexendoo commented Sep 26, 2023

Copy link
Copy Markdown
Member

r=me with rebase/squash

@kupiakos

kupiakos commented Dec 19, 2023

Copy link
Copy Markdown

Any chance this can be rebased and merged?

@cocodery cocodery mentioned this pull request Dec 28, 2023
Closed
@cocodery

cocodery commented Dec 28, 2023

Copy link
Copy Markdown
Contributor

I have tried locally, merging master branch and solving one small conflict will be fine.
And related Issue#12017 can tag fixed in the first comment.
@Centri3

@Alexendoo

Alexendoo commented Jan 5, 2024

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

📌 Commit 81f16d8 has been approved by Alexendoo

It is now in the queue for this repository.

@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit 81f16d8 with merge 43f39b5...

bors added a commit that referenced this pull request Jan 5, 2024
@bors
Auto merge of #10844 - Centri3:let_unit_value, r=Alexendoo
43f39b5 
Don't lint `let_unit_value` when `()` is explicit

since these are explicitly written (and not the result of a function call or anything else), they should be allowed, as they are both useful in some cases described in #9048

Fixes #9048

changelog: [`let_unit_value`]: Don't lint when `()` is explicit
@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-action_dev_test

@Alexendoo

Alexendoo commented Jan 5, 2024

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

📌 Commit fd9d330 has been approved by Alexendoo

It is now in the queue for this repository.

@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit fd9d330 with merge 394f63f...

@bors

bors commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Alexendoo
Pushing 394f63f to master...

@bors bors merged commit 394f63f into rust-lang:master Jan 5, 2024
as-com referenced this pull request in capnproto/capnproto-rust May 27, 2024
@as-com @dwrensha
Forbid use of as_slice when unaligned feature is enabled
1c57b0c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@llogiq llogiq llogiq left review comments

@Alexendoo Alexendoo Alexendoo approved these changes

+1 more reviewer

@danielhenrymantilla danielhenrymantilla danielhenrymantilla left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@llogiq llogiq

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

let_unit_value should be allowed when I explicitly write "let () = ..."

8 participants

@Centri3 @rustbot @bors @Alexendoo @kupiakos @cocodery @llogiq @danielhenrymantilla