Skip to content

Conversation

larsoner
Copy link
Contributor

@larsoner larsoner commented May 7, 2020

Not sure if you'd prefer a r string instead, but this fixes the char warning on nibabel import for me.

@codecov
Copy link

codecov bot commented May 7, 2020

Codecov Report

Merging #909 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
- Coverage   91.73%   91.73%   -0.01%     
==========================================
  Files          97       97              
  Lines       12359    12359              
  Branches     2177     2177              
==========================================
- Hits        11338    11337       -1     
  Misses        684      684              
- Partials      337      338       +1     
Impacted Files Coverage Δ
nibabel/cifti2/cifti2.py 96.67% <ø> (ø)
nibabel/volumeutils.py 83.94% <0.00%> (-0.20%) ⬇️

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 1660b1a...fcef511. Read the comment docs.

@matthew-brett
Copy link
Member

Really sorry - but would you consider an r string instead?

@larsoner
Copy link
Contributor Author

No problem, done!

@matthew-brett
Copy link
Member

Thanks - that's quick ...

@matthew-brett matthew-brett merged commit bfde527 into nipy:master May 13, 2020
@effigies effigies added this to the 3.2.0 milestone Aug 16, 2020
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.

3 participants