Skip to content

Conversation

@HansVRP
Copy link
Contributor

@HansVRP HansVRP commented Nov 7, 2025

No description provided.

@HansVRP HansVRP requested a review from soxofaan November 7, 2025 15:07
@HansVRP HansVRP force-pushed the issue744_download_optional branch from 8c95197 to 5206a43 Compare November 19, 2025 11:24
@HansVRP
Copy link
Contributor Author

HansVRP commented Nov 19, 2025

unclear why the unit tests are failing

@soxofaan
Copy link
Member

failing tests are because of #829 but that is fixed now on master, so make sure to merge master in this feature branch

@HansVRP HansVRP force-pushed the issue744_download_optional branch from 5206a43 to d90f06b Compare November 19, 2025 14:01
@HansVRP HansVRP requested a review from soxofaan November 19, 2025 14:05
@HansVRP
Copy link
Contributor Author

HansVRP commented Nov 19, 2025

@soxofaan rebased

soxofaan added a commit that referenced this pull request Nov 28, 2025
DummyBackend already covers all the mocking needs
soxofaan added a commit that referenced this pull request Nov 28, 2025
DummyBackend already covers all the mocking needs
@soxofaan soxofaan closed this Nov 28, 2025
@soxofaan soxofaan force-pushed the issue744_download_optional branch from a04316f to d0bb37e Compare November 28, 2025 12:33
@soxofaan soxofaan deleted the issue744_download_optional branch November 28, 2025 12:34
@soxofaan
Copy link
Member

finetuned the test
and merged in 9706528

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.

3 participants