Skip to content

Update Downloader interface to use suspend function#216

Merged
rfc2822 merged 3 commits intomainfrom
contact-downloader-suspending
Mar 10, 2026
Merged

Update Downloader interface to use suspend function#216
rfc2822 merged 3 commits intomainfrom
contact-downloader-suspending

Conversation

@rfc2822
Copy link
Member

@rfc2822 rfc2822 commented Mar 9, 2026

This pull request updates the Downloader interface to use a suspend function for coroutine support. It also adds KDoc comments for the interface and its download method.

Preparation for https://github.com/bitfireAT/davx5/issues/827 / bitfireAT/davx5-ose#2004

Changes:

  • Modified the download function to be a suspend function
  • Added KDoc comments for the Downloader interface and download method

- Add KDoc comments for Downloader interface and download method
- Change download function to suspend for coroutine support
@rfc2822 rfc2822 self-assigned this Mar 9, 2026
rfc2822 added 2 commits March 9, 2026 22:37
- Update `fromVCard`, `toContact`, and `getPhotoBytes` in `ContactReader` to suspend functions
- Update `fromReader` in `Contact` to suspend function
- Add `kotlinx-coroutines-test` dependency for testing
- Wrap test methods in `runTest` for coroutine support
@rfc2822 rfc2822 added refactoring Quality improvement of existing functions contacts labels Mar 9, 2026
@rfc2822 rfc2822 marked this pull request as ready for review March 10, 2026 09:32
@rfc2822 rfc2822 requested a review from a team as a code owner March 10, 2026 09:32
@rfc2822 rfc2822 merged commit e9ff994 into main Mar 10, 2026
7 checks passed
@rfc2822 rfc2822 deleted the contact-downloader-suspending branch March 10, 2026 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contacts refactoring Quality improvement of existing functions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant