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

Avoid repo type assumptions to enable proper tracking branch detection (master) #5128

Merged
merged 2 commits into from Nov 6, 2020

Conversation

mih
Copy link
Member

@mih mih commented Nov 5, 2020

This is the same as #5127 but we the fix ported to master.

Fixes gh-5107

Will fail on a crippled file system.
@mih mih changed the title Avoid repo type assumptions to enable proper tracking branch detection Avoid repo type assumptions to enable proper tracking branch detection (master) Nov 5, 2020
@codecov
Copy link

codecov bot commented Nov 5, 2020

Codecov Report

Merging #5128 into master will decrease coverage by 0.02%.
The diff coverage is 81.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5128      +/-   ##
==========================================
- Coverage   89.58%   89.56%   -0.03%     
==========================================
  Files         299      299              
  Lines       42075    42085      +10     
==========================================
  Hits        37692    37692              
- Misses       4383     4393      +10     
Impacted Files Coverage Δ
datalad/core/distributed/tests/test_clone.py 93.98% <77.77%> (-0.25%) ⬇️
datalad/support/gitrepo.py 90.48% <100.00%> (+<0.01%) ⬆️
datalad/downloaders/tests/test_http.py 88.65% <0.00%> (-1.90%) ⬇️

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 a0b3f3b...79c79b2. Read the comment docs.

…ction

Force-treating a to-be-added submodule as a GitRepo prevented the
detection of a corresponding branch to the active branch, in order to
determine the true tracking branch. Which is necessary to discover
the effective clone URL from a repository (when it is in adjusted
mode).

Fixes dataladgh-5107
@mih mih merged commit cec5f0c into datalad:master Nov 6, 2020
2 of 4 checks passed
@mih mih deleted the bf-5107 branch November 6, 2020 12:11
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.

Tracking branch logic makes platform homogeneity assumptions
1 participant