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

Adds whitespace, to appease the sphinx. #1508

Merged
merged 3 commits into from May 4, 2018

Conversation

Projects
None yet
4 participants
@arokem
Copy link
Member

arokem commented May 3, 2018

This should at least partially address #1507

DOC: add some extra indentation, fixes
Adds stuff to Ariel's PR, allowing me to close mine (#1508).
@matthew-brett

This comment has been minimized.

Copy link
Member

matthew-brett commented May 3, 2018

OK - I added the stuff in my PR with arokem#30 - closing mine.

Merge pull request #30 from matthew-brett/add-my-fixes-to-ariels
Pull in extra stuff from my doc PR
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented May 3, 2018

Codecov Report

Merging #1508 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1508   +/-   ##
=======================================
  Coverage   87.55%   87.55%           
=======================================
  Files         246      246           
  Lines       31366    31366           
  Branches     3433     3433           
=======================================
  Hits        27463    27463           
  Misses       3085     3085           
  Partials      818      818
Impacted Files Coverage Δ
dipy/reconst/dki.py 96.82% <ø> (ø) ⬆️

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 435f47a...900de02. Read the comment docs.

@skoudoro

This comment has been minimized.

Copy link
Member

skoudoro commented May 4, 2018

Thank you @arokem and @matthew-brett

I made a quick tour of the documentation and it seems to be the only place with this problem.

merging

@skoudoro skoudoro merged commit e8e588a into nipy:master May 4, 2018

3 checks passed

codecov/patch Coverage not affected when comparing 435f47a...900de02
Details
codecov/project 87.55% remains the same compared to 435f47a
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

ShreyasFadnavis pushed a commit to ShreyasFadnavis/dipy that referenced this pull request Sep 20, 2018

DOC: add some extra indentation, fixes
Adds stuff to Ariel's PR, allowing me to close mine (nipy#1508).

ShreyasFadnavis pushed a commit to ShreyasFadnavis/dipy that referenced this pull request Sep 20, 2018

Merge pull request nipy#1508 from arokem/math-in-kurtosis
Adds whitespace, to appease the sphinx.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment