Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

process: remove lock/unlock #105

Merged
merged 1 commit into from
Jan 13, 2023
Merged

process: remove lock/unlock #105

merged 1 commit into from
Jan 13, 2023

Conversation

skshetry
Copy link
Member

@skshetry skshetry commented Jan 12, 2023

@skshetry skshetry requested a review from karajan1001 January 12, 2023 10:29
@codecov-commenter
Copy link

Codecov Report

Base: 78.18% // Head: 78.19% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (247c244) compared to base (2665162).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   78.18%   78.19%   +0.01%     
==========================================
  Files          21       21              
  Lines         935      931       -4     
  Branches      148      147       -1     
==========================================
- Hits          731      728       -3     
+ Misses        176      175       -1     
  Partials       28       28              
Impacted Files Coverage Δ
src/dvc_task/proc/process.py 83.47% <100.00%> (-0.53%) ⬇️
src/dvc_task/worker/temporary.py 81.96% <0.00%> (+1.63%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pmrowla pmrowla merged commit 643ce11 into main Jan 13, 2023
@pmrowla pmrowla deleted the remove-lock-unlock branch January 13, 2023 08:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants