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

Fixes for RIA when using different push/pull URLs #5608

Merged
merged 3 commits into from Apr 28, 2021
Merged

Conversation

mih
Copy link
Member

@mih mih commented Apr 28, 2021

Sister PR of #5607 against master

Additionally alters the behavior of the new alias feature introduced with #5592 to use a relative path for the alias. This keep the symlinks valid when moving an entire store.

mih added 2 commits April 28, 2021 11:04
This also fixes a problem with the version check for a newly created
dataset, which would perform a local test, even if the sibling is
on an SSH host.
@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #5608 (e0584f2) into master (bc3f743) will decrease coverage by 13.44%.
The diff coverage is 92.85%.

❗ Current head e0584f2 differs from pull request most recent head aabd90e. Consider uploading reports for the commit aabd90e to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           master    #5608       +/-   ##
===========================================
- Coverage   90.33%   76.89%   -13.45%     
===========================================
  Files         305      302        -3     
  Lines       41645    41609       -36     
===========================================
- Hits        37622    31995     -5627     
- Misses       4023     9614     +5591     
Impacted Files Coverage Δ
datalad/distributed/ora_remote.py 31.40% <0.00%> (-0.41%) ⬇️
datalad/customremotes/ria_utils.py 93.54% <100.00%> (-0.11%) ⬇️
datalad/plugin/wtf.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/addurls.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/__init__.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/no_annex.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/add_readme.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/check_dates.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/support/tests/utils.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/plugin/export_archive.py 0.00% <0.00%> (-100.00%) ⬇️
... and 126 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 bc3f743...aabd90e. Read the comment docs.

This must be executed on the remote end, or a success is not
informative.
@mih mih changed the title RF: Consolidate RIA versions checks into a single helper Fixes for RIA when using different push/pull URLs Apr 28, 2021
@bpoldrack bpoldrack merged commit 485b769 into datalad:master Apr 28, 2021
@mih mih deleted the bf-ria branch April 28, 2021 13:07
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