-
Notifications
You must be signed in to change notification settings - Fork 82
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 broken links in docs #1017
[DOC] Fix some broken links in docs #1017
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @NicolasGensollen ! some very needed changes in there !
I have a somewhat opposite view to what you did in the matter of the links you've given for BIDS and other references. I feel there is too many of them, and the text might look like a Christmas tree, which may not be so nice for the readers ? WDYT ? Opened to changing my mind !
d3fbfc9
to
c70696c
Compare
Thanks for the review @MatthieuJoulot ! I realized there is already a list of converters on the index page which was already up-to-date. I made a link to this list instead of copy-pasting it multiple times in the "Interacting with clinica" page. I also removed the multiple occurrences of links as you suggested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !
Fix some broken links and add some new links to the documentation.