Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix publish-depends for ORA remotes in clone (in maint this time) #5488

Merged
merged 1 commit into from Mar 12, 2021

Conversation

mih
Copy link
Member

@mih mih commented Mar 12, 2021

This is #5415 but against maint

Setting the publication dependency for 'origin' on an ORA remote in
postclone routine didn't properly account for multiple accessible ORA
remotes or the fact that none of them may be associated with the RIA
store we are cloning from.

Now, first check for an ORA remote with the same store URL registered
and if there is none check whether the store has a remote's uuid stored
in the .git/config of the dataset. If we have that remote enabled, set
the dependency on it.

(Closes datalad#5396)
@mih mih changed the base branch from master to maint March 12, 2021 06:09
@codecov
Copy link

codecov bot commented Mar 12, 2021

Codecov Report

Merging #5488 (a8ba155) into maint (15de1fb) will decrease coverage by 0.60%.
The diff coverage is 95.65%.

Impacted file tree graph

@@            Coverage Diff             @@
##            maint    #5488      +/-   ##
==========================================
- Coverage   90.15%   89.55%   -0.61%     
==========================================
  Files         296      296              
  Lines       41796    41824      +28     
==========================================
- Hits        37683    37454     -229     
- Misses       4113     4370     +257     
Impacted Files Coverage Δ
datalad/core/distributed/clone.py 90.93% <92.00%> (+0.02%) ⬆️
datalad/core/distributed/tests/test_clone.py 96.95% <100.00%> (-0.35%) ⬇️
datalad/support/due.py 48.00% <0.00%> (-32.00%) ⬇️
datalad/ui/utils.py 56.25% <0.00%> (-21.88%) ⬇️
datalad/metadata/extractors/tests/test_base.py 87.80% <0.00%> (-12.20%) ⬇️
datalad/metadata/extractors/tests/test_xmp.py 88.46% <0.00%> (-11.54%) ⬇️
datalad/metadata/extractors/tests/test_image.py 88.46% <0.00%> (-11.54%) ⬇️
datalad/support/archive_utils_patool.py 29.87% <0.00%> (-10.39%) ⬇️
datalad/metadata/tests/test_extract_metadata.py 91.11% <0.00%> (-8.89%) ⬇️
datalad/metadata/extractors/tests/test_exif.py 92.00% <0.00%> (-8.00%) ⬇️
... and 56 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 15de1fb...a8ba155. Read the comment docs.

@bpoldrack bpoldrack merged commit 8d24773 into datalad:maint Mar 12, 2021
14 of 15 checks passed
@mih mih deleted the bf-5396 branch March 19, 2021 11:44
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.

None yet

2 participants