Skip to content

Issue template: Pretty print platform details #2671

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 2 commits into from
Aug 2, 2018
Merged

Issue template: Pretty print platform details #2671

merged 2 commits into from
Aug 2, 2018

Conversation

effigies
Copy link
Member

@effigies effigies commented Aug 2, 2018

This will help make issues more readable by adding newlines and vertically aligning keys. Pretty-printing is a built-in, so no dependencies are needed.

I've also moved the suggestion to an HTML comment and provided pre-written triple ticks to help with formatting for new users.

This will help make issues more readable by adding newlines and vertically aligning keys. Pretty-printing is a built-in, so no dependencies are needed.

I've also moved the suggestion to an HTML comment and provided pre-written triple ticks to help with formatting for new users.
@@ -12,7 +12,13 @@ Please put URL to code or code here (if not too long).

### Platform details:

Please paste the output of: `python -c "import nipype; print(nipype.get_info()); print(nipype.__version__)"`
<!-- Please run the following code from your shell and place the output between the triple ticks, below.
python -c "import nipype; from pprint import pprint; pprint(nipype.get_info()); print(nipype.__version__)"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since nipype's version is included in get_info, we could also remove the print(nipype.__version__) call.

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2671      +/-   ##
==========================================
- Coverage   67.62%   65.08%   -2.54%     
==========================================
  Files         340      340              
  Lines       43056    43056              
  Branches     5329     5329              
==========================================
- Hits        29116    28025    -1091     
- Misses      13238    13976     +738     
- Partials      702     1055     +353
Flag Coverage Δ
#smoketests ?
#unittests 65.08% <ø> (ø) ⬆️
Impacted Files Coverage Δ
nipype/algorithms/rapidart.py 35.39% <0%> (-29.21%) ⬇️
nipype/utils/provenance.py 55.73% <0%> (-28.99%) ⬇️
nipype/interfaces/fsl/model.py 55.26% <0%> (-25.35%) ⬇️
nipype/interfaces/spm/base.py 67.98% <0%> (-19.48%) ⬇️
nipype/workflows/fmri/fsl/preprocess.py 72.13% <0%> (-13.94%) ⬇️
nipype/pipeline/plugins/multiproc.py 72.15% <0%> (-12.66%) ⬇️
nipype/workflows/fmri/spm/preprocess.py 60.2% <0%> (-12.25%) ⬇️
nipype/interfaces/fsl/preprocess.py 72.7% <0%> (-9.91%) ⬇️
nipype/interfaces/matlab.py 76.76% <0%> (-9.1%) ⬇️
nipype/algorithms/modelgen.py 58.83% <0%> (-8.61%) ⬇️
... and 29 more

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 69dce12...18b5134. Read the comment docs.

@effigies effigies added this to the 1.1.2 milestone Aug 2, 2018
@mgxd mgxd merged commit 2e69844 into master Aug 2, 2018
@effigies effigies deleted the gh/prettyprint branch August 2, 2018 17:20
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