Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Fix unit tests #57

Closed
1 task
kabilar opened this issue Aug 5, 2024 · 4 comments
Closed
1 task

Fix unit tests #57

kabilar opened this issue Aug 5, 2024 · 4 comments

Comments

@kabilar
Copy link
Member

kabilar commented Aug 5, 2024

  • Requires publishing Archive Docker image for the Client to run against
@kabilar
Copy link
Member Author

kabilar commented Oct 29, 2024

Hi @aaronkanzer, how difficult would this be to tackle as part of our cleanup sprint?

@aaronkanzer
Copy link

Hi @kabilar -- I think with the diff we have on checking for authentication on every call, I'd like to probably punt this issue until we develop a way to integrate our API back into the dandi-cli tool -- let me know though if you feel strongly either way

@kabilar
Copy link
Member Author

kabilar commented Oct 29, 2024

Thanks Aaron. That's a good point. This would be redundant work if the dandi-cli was abstracted in a way to incorporate the changes with the lincbrain-cli.

@kabilar
Copy link
Member Author

kabilar commented Nov 13, 2024

Closing as we are working to replace the lincbrain-cli with the dandi-cli. See dandi#1519.

@kabilar kabilar closed this as not planned Won't fix, can't repro, duplicate, stale Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants