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.

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

[DOC] Fix some links [ci skip] #3054

Merged
merged 2 commits into from
Feb 7, 2024
Merged

[DOC] Fix some links [ci skip] #3054

merged 2 commits into from
Feb 7, 2024

Conversation

skoudoro
Copy link
Member

@skoudoro skoudoro commented Feb 1, 2024

This small PR is to fix some failing URLs in the current documentation.

Copy link
Contributor

@arokem arokem left a comment

Choose a reason for hiding this comment

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

Small edit suggestion

doc/conf.py Outdated Show resolved Hide resolved
Co-authored-by: Ariel Rokem <arokem@gmail.com>
@skoudoro
Copy link
Member Author

skoudoro commented Feb 1, 2024

Thanks ! I forgot the skip ci during the commit

Copy link

codecov bot commented Feb 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fb4005f) 82.39% compared to head (916757c) 82.37%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3054      +/-   ##
==========================================
- Coverage   82.39%   82.37%   -0.02%     
==========================================
  Files         146      146              
  Lines       20631    20631              
  Branches     3319     3319              
==========================================
- Hits        16999    16995       -4     
- Misses       2802     2805       +3     
- Partials      830      831       +1     

see 1 file with indirect coverage changes

@skoudoro skoudoro merged commit 6f24685 into dipy:master Feb 7, 2024
27 checks passed
@skoudoro skoudoro deleted the doc-fix branch February 7, 2024 14:05
@skoudoro
Copy link
Member Author

skoudoro commented Feb 7, 2024

No additional comments so merging

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.

None yet

2 participants