[FIX] Remove yaml list key for API_QUERY_URL

[surchs]

Closes #163

Changes proposed in this pull request:

• remove the dash

Possibly still to be done:

• confirm this is actually a yaml-list (i.e. - dash) issue
• can we test this so it doesn't happen again?

Checklist

• PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
• PR links to GitHub issue with mention Closes #XXXX
• Tests pass
• Checks pass

For new features:

• Tests have been added

For bug fixes:

• There is at least one test that would fail under the original bug conditions.

[alyssadai]

Thanks for the quick fix @surchs 💪

Re:

can we test this so it doesn't happen again?

The easiest solution would probably be to add to our pre-commit config a hook to validate the docker compose config file itself, i.e.

• docker-compose-check - https://github.com/IamTheFij/docker-pre-commit (also listed under https://pre-commit.com/hooks.html)

[surchs]

Thanks for the quick fix @surchs 💪

Re:

can we test this so it doesn't happen again?

The easiest solution would probably be to add to our pre-commit config a hook to validate the docker compose config file itself, i.e.

• docker-compose-check - https://github.com/IamTheFij/docker-pre-commit (also listed under https://pre-commit.com/hooks.html)

Yeah I think that's a really good idea, could you make an issue and then we address that maybe for the dsst milestone?