Skip to content

[FIX] Ensure non-nifti resources get downloaded when downloading session as a zip#283

Merged
DESm1th merged 2 commits intoTIGRLab:masterfrom
DESm1th:fetch_fix
Aug 7, 2020
Merged

[FIX] Ensure non-nifti resources get downloaded when downloading session as a zip#283
DESm1th merged 2 commits intoTIGRLab:masterfrom
DESm1th:fetch_fix

Conversation

@DESm1th
Copy link
Copy Markdown
Contributor

@DESm1th DESm1th commented Aug 6, 2020

No description provided.

@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Aug 6, 2020

Hello @DESm1th, Thank you for updating!

Cheers! There are no style issues detected in this Pull Request. 🍻 To test for issues locally, pip install flake8 and then run flake8 datman.

Comment last updated at 2020-08-06 21:48:23 UTC

@auto-assign auto-assign bot requested review from edickie, jerdra and josephmje August 6, 2020 21:47
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 6, 2020

Codecov Report

Merging #283 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
- Coverage   31.88%   31.87%   -0.01%     
==========================================
  Files          57       57              
  Lines        8679     8680       +1     
==========================================
  Hits         2767     2767              
- Misses       5912     5913       +1     
Impacted Files Coverage Δ
datman/xnat.py 27.84% <0.00%> (-0.04%) ⬇️

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 93c2d7a...6f21c8c. Read the comment docs.

@DESm1th DESm1th merged commit 1528e9b into TIGRLab:master Aug 7, 2020
@DESm1th DESm1th deleted the fetch_fix branch August 7, 2020 15:23
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.

4 participants