Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

nes: fix: don't yield edits when fetch errors or is cancelled#4869

Merged
eleanorjboyd merged 3 commits into
release/0.42from
ulugbekna/nes-dont-yield-fetch-with-error-candidate
Mar 31, 2026
Merged

nes: fix: don't yield edits when fetch errors or is cancelled#4869
eleanorjboyd merged 3 commits into
release/0.42from
ulugbekna/nes-dont-yield-fetch-with-error-candidate

Conversation

@ulugbekna

@ulugbekna ulugbekna commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

@ulugbekna

Copy link
Copy Markdown
Contributor Author

don't mind -- these're failing because I first incorrectly created the PR against main, but also they're not required and needed

CleanShot 2026-03-31 at 20 16 12@2x

@ulugbekna ulugbekna enabled auto-merge (squash) March 31, 2026 15:19

@alexr00 alexr00 left a comment

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.

Great to see the tests. Thank you!

@alexr00 alexr00 closed this Mar 31, 2026
auto-merge was automatically disabled March 31, 2026 15:51

Pull request was closed

@alexr00 alexr00 reopened this Mar 31, 2026
@alexr00 alexr00 enabled auto-merge March 31, 2026 16:26
auto-merge was automatically disabled March 31, 2026 16:48

Pull request was closed

@eleanorjboyd eleanorjboyd reopened this Mar 31, 2026
@eleanorjboyd eleanorjboyd merged commit 9a12b5d into release/0.42 Mar 31, 2026
29 of 30 checks passed
@eleanorjboyd eleanorjboyd deleted the ulugbekna/nes-dont-yield-fetch-with-error-candidate branch March 31, 2026 17:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants