Skip to content

Implement next(isolation:) in AsyncAdjacentPairsSequence#390

Open
alex-waldron wants to merge 4 commits intoapple:mainfrom
alex-waldron:implement-isolated-next-adjacent-pairs
Open

Implement next(isolation:) in AsyncAdjacentPairsSequence#390
alex-waldron wants to merge 4 commits intoapple:mainfrom
alex-waldron:implement-isolated-next-adjacent-pairs

Conversation

@alex-waldron
Copy link
Copy Markdown

The next(isolation:) method allows the iterator to inherit the caller's isolation context, avoiding unnecessary isolation hops.

The next(isolation:) method allows the iterator to inherit the caller's
isolation context, avoiding unnecessary isolation hops.
@alex-waldron alex-waldron force-pushed the implement-isolated-next-adjacent-pairs branch from 1ca8823 to 0f896c5 Compare December 27, 2025 21:09
@alex-waldron alex-waldron marked this pull request as ready for review January 10, 2026 12:13

import XCTest
import AsyncAlgorithms
import Observation
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think the test does not really need Observations to work - we have other things that dont have this added dependency.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Nope it doesn't. Its just what I was using when I discovered the issue. I replaced it with an existing test async sequence in 425dba6

@alex-waldron alex-waldron requested a review from phausler April 14, 2026 12:20
@phausler
Copy link
Copy Markdown
Member

@swift-ci please test

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