search

laminlabs / lamindb-setup

Setup & configure LaminDB.
Apache License 2.0
4 stars 1 forks source link

🐛 Yet another fix for parallel import #708

Closed Koncopd closed 4 months ago

falexwolf commented 4 months ago

@Koncopd - don't you agree we need a test; otherwise, we'll keep missing things. I'm just finishing the whole typing issues, that's why I haven't started yet.

Koncopd commented 4 months ago

Yep, absolutely agree that we need a test and that it should be in lamindb.

github-actions[bot] commented 4 months ago

🚀 Deployed on https://66101d786670f50e45c72a5a--lamindb-setup-htry.netlify.app

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 82.65%. Comparing base (137843a) to head (64d384a). Report is 1 commits behind head on main.

Files Patch % Lines
lamindb_setup/core/django.py 50.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #708 +/- ## ========================================== - Coverage 82.71% 82.65% -0.06% ========================================== Files 39 39 Lines 2494 2497 +3 ========================================== + Hits 2063 2064 +1 - Misses 431 433 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.