datalad / datalad-neuroimaging

DataLad extension for neuroimaging research
http://datalad.org
Other
17 stars 14 forks source link

BF:Docs: replace incorrect dashes with spaces in command names #90

Closed loj closed 2 years ago

loj commented 3 years ago

The documentation for the commands contain dashes that should be spaces, ie:

datalad-bids2scidata should be datalad bids2scidata

This applies a fix that was made for other docs (https://github.com/datalad/datalad/pull/1761).

(Closes #89)

jsheunis commented 2 years ago

This looks like an easy merge. Anyone else want to comment? @yarikoptic @bpoldrack?

Regarding the appveyor build failure, the urllib.error.HTTPError: HTTP Error 404: Not Found error during download of datalad_installer (datalad_installer Downloading http://ftp.us.debian.org/debian/pool/main/g/git-annex/git-annex_8.20201127-1_amd64.deb) occurs in #88 as well. Is this expected? Should it be a separate issue?

mslw commented 2 years ago

The build failure happened 8 months ago. From what I can see, the appveyor configuration has been updated since in d0a3097ee71d40313536d7cba4adbc0994571b4d to use a newer version of git-annex (reflecting datalad core), so there's a chance the test would just complete if run now.

There might, however, be new problems with testing, in light of #91

mslw commented 2 years ago

I triggered a re re-run of the tests. With past and recent updates to appveyor config, the tests failure is out of the way.