Skip to content

FIX: Typo in private method name #1978

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

Merged
merged 1 commit into from
Apr 28, 2017
Merged

FIX: Typo in private method name #1978

merged 1 commit into from
Apr 28, 2017

Conversation

effigies
Copy link
Member

#1966 bug.

Sorry.

@effigies
Copy link
Member Author

Testing locally. Will post when I'm sure this fixes the problem.

@effigies
Copy link
Member Author

Fixes the issue. ReconAll runs fine with and without existing expert options.

@codecov-io
Copy link

codecov-io commented Apr 28, 2017

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1978   +/-   ##
======================================
  Coverage    72.6%   72.6%           
======================================
  Files        1070    1070           
  Lines       54456   54456           
  Branches     7868    7868           
======================================
  Hits        39537   39537           
  Misses      13686   13686           
  Partials     1233    1233
Flag Coverage Δ
#smoketests 72.6% <ø> (ø) ⬆️
#unittests 70.15% <ø> (ø) ⬆️
Impacted Files Coverage Δ
nipype/interfaces/freesurfer/preprocess.py 65.28% <ø> (ø) ⬆️

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 a17af6b...c480ef0. Read the comment docs.

@satra satra merged commit 5a88840 into master Apr 28, 2017
@effigies effigies deleted the fix/expert_options_typo branch April 28, 2017 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants