Skip to content

🔧 Updated documentation for collect function regarding isolate groups (#533) #2032

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

Closed
wants to merge 2 commits into from

Conversation

Victowolf
Copy link

@Victowolf Victowolf commented Mar 7, 2025

  • Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

🔧 Updated documentation for collect function regarding isolate groups (#533)

Pull Request Description

This PR updates the documentation for the collect function to provide better clarity on how it handles isolate groups. The changes aim to:

✅ Improve readability and maintainability of the documentation.
✅ Clearly explain the behavior of collect in scenarios involving isolate groups.
✅ Address issue #533 by enhancing the function’s inline comments and docstrings.

Changes Made:

📌 Updated lib/src/collect.dart with improved documentation.

Checklist:

  • Documentation updated
  • Code follows project guidelines
  • Ready for review

Copy link
Contributor

@liamappelbe liamappelbe left a comment

Choose a reason for hiding this comment

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

The PR includes a bunch of unrelated changes. Looks like it got mixed up with #2029

Also, needs to be synced to head

@Victowolf Victowolf closed this by deleting the head repository Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants