Skip to content
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

Merge preparation for amikai's itereach PR #489

Merged
merged 8 commits into from
Sep 6, 2024
Merged

Conversation

mna
Copy link
Member

@mna mna commented Sep 6, 2024

amikai and others added 8 commits July 20, 2024 11:37
- We don't need to use the iter.Seq2 type; instead, we can use func(yield func(int, *Selection) bool). In fact, the underlying type of iter.Seq2 is func(yield func(int, *Selection) bool), so we don't need to upgrade the go.mod version.
- For for-range testing in version 1.23, we can use build tags to compile only for versions above 1.23.
@mna mna merged commit 9551ee5 into master Sep 6, 2024
6 checks passed
@andig
Copy link

andig commented Sep 7, 2024

Sorry if I'm nitpicking here.
I'm not sure if user an iter.Seq2 is the right implementation choice. Afaiaa the index part of the iteration is already provided by the range statement itself. The iterator here should be an iter.Seq if I'm not mistaken. Using Seq2 just makes it confusing since that would usually be used with returning errors.
I may of course also be completely wrong.

update nvm- I'm wrong. Sorry for the noise.

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.

3 participants