Skip to content

check reverse inference of closure return type for zip_with#120

Merged
MusicalNinjaDad merged 1 commit into
mainfrom
reverse_inference
Jun 2, 2026
Merged

check reverse inference of closure return type for zip_with#120
MusicalNinjaDad merged 1 commit into
mainfrom
reverse_inference

Conversation

@MusicalNinjaDad
Copy link
Copy Markdown
Owner

No description provided.

@MusicalNinjaDad MusicalNinjaDad enabled auto-merge (squash) June 2, 2026 13:32
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • In the new test, consider using |_, _| () instead of Default::default() to make the intended unit return type explicit and avoid relying on Default for clarity.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- In the new test, consider using `|_, _| ()` instead of `Default::default()` to make the intended unit return type explicit and avoid relying on `Default` for clarity.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@MusicalNinjaDad MusicalNinjaDad merged commit 7964522 into main Jun 2, 2026
19 checks passed
@MusicalNinjaDad MusicalNinjaDad deleted the reverse_inference branch June 2, 2026 13:33
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.

1 participant