Skip to content

[DOC] Fixed Developer Setup Link in install.rst #3330

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 3 commits into from
Apr 22, 2021

Conversation

RaunakJalan
Copy link
Contributor

Summary

Changed the link for developer setup in install.rst to make it working. Previously showing Error 404.
Now the link is working and redirecting to appropriate page.

Developer Install Page

List of changes proposed in this PR (pull-request)

  • Change link of developer setup to <../devel/testing_nipype.rst> instead of <../devel/testing_nipype.html>

Acknowledgment

  • (Mandatory) I acknowledge that this contribution will be available under the Apache 2 license.

@codecov
Copy link

codecov bot commented Apr 22, 2021

Codecov Report

Merging #3330 (dc57459) into master (89d828c) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3330      +/-   ##
==========================================
+ Coverage   65.05%   65.09%   +0.04%     
==========================================
  Files         302      302              
  Lines       40027    40027              
  Branches     5292     5292              
==========================================
+ Hits        26038    26055      +17     
+ Misses      12916    12904      -12     
+ Partials     1073     1068       -5     
Flag Coverage Δ
unittests 65.01% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
nipype/pipeline/plugins/base.py 59.72% <0.00%> (+1.64%) ⬆️
nipype/pipeline/plugins/legacymultiproc.py 71.07% <0.00%> (+5.39%) ⬆️

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 89d828c...dc57459. Read the comment docs.

@effigies
Copy link
Member

Great, thanks! Would you like to add yourself to the .zenodo.json file?

@RaunakJalan
Copy link
Contributor Author

RaunakJalan commented Apr 22, 2021

Yeah sure @effigies. I would like to add myself to it. Looking to make more contributions in future.

@effigies
Copy link
Member

Sounds great! Just add your information after this entry: https://github.com/RaunakJalan/nipype/blob/doc_issue/.zenodo.json#L796-L800

@effigies effigies changed the title [FIX]:#3329 Fixed Developer Setup Link in install.rst [DOC] Fixed Developer Setup Link in install.rst Apr 22, 2021
@effigies effigies merged commit 4e8f54d into nipy:master Apr 22, 2021
@effigies effigies added this to the 1.6.1 milestone May 1, 2021
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.

2 participants