Skip to content

Conversation

ash2k
Copy link
Contributor

@ash2k ash2k commented Sep 14, 2025

Both inner and outer for must be exited when yield returns false.

@rueian rueian merged commit 8aa9c89 into redis:main Sep 14, 2025
34 checks passed
@rueian
Copy link
Collaborator

rueian commented Sep 14, 2025

Thank you @ash2k

@ash2k ash2k deleted the fix-scanner-early-exit branch September 15, 2025 00:58
@ash2k
Copy link
Contributor Author

ash2k commented Sep 15, 2025

@rueian No problem, thanks for merging. Could you make a release please? I'm working on a refactoring (in my project) and that's how I found the bug. Now I need the fix to continue working on it :)

@rueian
Copy link
Collaborator

rueian commented Sep 15, 2025

Yes, I will release a new version this week!

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.

2 participants