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

Fix typo in iterate(itr::AsyncCollector) #43041

Merged
merged 3 commits into from
Nov 17, 2021
Merged

Conversation

pfitzseb
Copy link
Member

No idea how to test this, I just stumbled over a linter warning for the undeclared f.

@vchuravy vchuravy added the needs tests Unit tests are required for this change label Nov 11, 2021
base/asyncmap.jl Outdated Show resolved Hide resolved
@aviatesk
Copy link
Sponsor Member

Tests could be added later by someone who uses this features. I guess we may also want to improve the general code quality here (e.g. I don't see why we need to set AsyncCollector to be mutable).

@aviatesk aviatesk merged commit 89c137c into JuliaLang:master Nov 17, 2021
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs tests Unit tests are required for this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants