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.

Sign up for GitHub

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

Jump to bottom

[DOC] Fix wrong indentation of example code #4053

Merged
merged 1 commit into from
Oct 13, 2023
Merged

[DOC] Fix wrong indentation of example code #4053

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
21 changes: 9 additions & 12 deletions CONTRIBUTING.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -414,8 +414,7 @@ Code inside ``maskers._validation.py``:
to make it part of the maskers module's public API
"""


return _check_mask_shape(mask_img) and _check_mask_values(mask_img)
return _check_mask_shape(mask_img) and _check_mask_values(mask_img)


def _check_mask_shape(mask_img):
Expand All @@ -433,19 +432,17 @@ Code inside ``maskers._validation.py``:
class _Validator:
"""Private internal of the _validation module"""

def validate(self, img):
"""Public API of _Validator"""

def validate(self, img):
"""Public API of _Validator"""

def _validate_shape(self, img):
"""Private internal of the _Validator class.

def _validate_shape(self, img):
"""Private internal of the _Validator class.
As we don't use the double leading underscore in nilearn we
cannot infer from the name alone if it is considered to be
exposed to subclasses or not.

As we don't use the double leading underscore in nilearn we cannot
infer from the name alone if it is considered to be exposed to
subclasses or not.

"""
"""

..
Source: Jerome Dockes https://github.com/nilearn/nilearn/issues/3628#issuecomment-1515211711
Expand Down