search

psychoinformatics-de / inm-icf-utilities

MIT License
0 stars 5 forks source link

Remove (subjectively) superfluous colon #28

Closed adswa closed 1 year ago

adswa commented 1 year ago

If anyone likes it, feel free to close this PR :)

adswa commented 1 year ago

Test failure seems unrelated:

FAILED ../tests/test_datalad_workflows/test_pipeline.py::test_pipeline - datalad.support.exceptions.IncompleteResultsError: Command did not complete successfully. 1 failed:
[{'action': 'get',
  'annexkey': 'MD5E-s4720640--dc6af77bd347c355763255e0d985b325.tar',
  'error_message': 'external special remote error: '
                   "HTTPSConnectionPool(host='data.inm-icf.de', port=443): Max "
                   'retries exceeded with url: /study_1/visit_a_dicom.tar '
                   '(Caused by '
                   "NewConnectionError('<urllib3.connection.HTTPSConnection "
                   'object at 0x7f09425e9e80>: Failed to establish a new '
                   "connection: [Errno 111] Connection refused'))",
  'path': '/tmp/pytest-of-appveyor/pytest-0/test_pipeline0/ds_study_1_visit_a/icf/visit_a_dicom.tar',
  'refds': '/tmp/pytest-of-appveyor/pytest-0/test_pipeline0/ds_study_1_visit_a',
  'status': 'error',
  'type': 'file'}]
mih commented 1 year ago

I believe these tests are meant to fail. I disabled that part in my #26 to see whether the rest is working