Skip to content

chore: Replace ioCoroutineContext where it's not needed#2803

Draft
LouisCAD wants to merge 2 commits intomainfrom
replace-io-coroutine-context
Draft

chore: Replace ioCoroutineContext where it's not needed#2803
LouisCAD wants to merge 2 commits intomainfrom
replace-io-coroutine-context

Conversation

@LouisCAD
Copy link
Copy Markdown
Contributor

@LouisCAD LouisCAD commented Feb 4, 2026

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 4, 2026

@LouisCAD LouisCAD disabled auto-merge March 12, 2026 12:12
@LouisCAD
Copy link
Copy Markdown
Contributor Author

After review with @benjaminVadon and @sirambd, our coroutineContext function was sneakily also adding a CoroutineExceptionHandler that reports uncaught exceptions (and ignores some Realm-related ones), without letting the app crash.

We need to change the naming/API of this, and also improve it to have helpful reports over Sentry.

@LouisCAD LouisCAD marked this pull request as draft March 12, 2026 12:21
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.

1 participant