Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 5, 2025

Replaces RxJS-based retry logic in execAndWaitForOutputToMatch with standard async/await for better readability and reduced dependency footprint.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 5, 2025
@clydin clydin force-pushed the e2e/remove-rxjs-utils branch from f81fe40 to d9325f4 Compare December 5, 2025 17:20
@clydin clydin marked this pull request as ready for review December 5, 2025 17:32
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Dec 5, 2025
@clydin clydin requested a review from alan-agius4 December 5, 2025 17:37
Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

Replaces RxJS-based retry logic in `execAndWaitForOutputToMatch` with
standard async/await for better readability and reduced dependency footprint.
@clydin clydin force-pushed the e2e/remove-rxjs-utils branch from d9325f4 to a34a045 Compare December 5, 2025 17:42
@clydin clydin requested a review from alan-agius4 December 5, 2025 17:42
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 5, 2025
@alan-agius4 alan-agius4 removed their request for review December 5, 2025 17:42
@clydin clydin merged commit 78f9013 into angular:main Dec 5, 2025
35 checks passed
@clydin
Copy link
Member Author

clydin commented Dec 5, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the e2e/remove-rxjs-utils branch December 5, 2025 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants