search

audeering / audbackend

Manage file storage on different backends
https://audeering.github.io/audbackend/
Other
3 stars 0 forks source link

Remove audbackend.available() #202

Closed hagenw closed 3 months ago

hagenw commented 3 months ago

Relates to #197

This pull request proposes to remove audbackend.available() as we cannot maintain its current behavior/results with the new approach of accessing backends by instantiating them, introduced in #200.

There might be ways to preserve its original behavior, but I don't think this is worse the effort.

I also updated audbackend/core/conftest.py as proposed in https://github.com/audeering/audbackend/pull/200#issuecomment-2024615148.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.0%. Comparing base (44c4e7e) to head (011208c).

Additional details and impacted files | [Files](https://app.codecov.io/gh/audeering/audbackend/pull/202?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering) | Coverage Δ | | |---|---|---| | [audbackend/\_\_init\_\_.py](https://app.codecov.io/gh/audeering/audbackend/pull/202?src=pr&el=tree&filepath=audbackend%2F__init__.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering#diff-YXVkYmFja2VuZC9fX2luaXRfXy5weQ==) | `100.0% <ø> (ø)` | | | [audbackend/core/api.py](https://app.codecov.io/gh/audeering/audbackend/pull/202?src=pr&el=tree&filepath=audbackend%2Fcore%2Fapi.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering#diff-YXVkYmFja2VuZC9jb3JlL2FwaS5weQ==) | `100.0% <ø> (ø)` | | | [audbackend/core/conftest.py](https://app.codecov.io/gh/audeering/audbackend/pull/202?src=pr&el=tree&filepath=audbackend%2Fcore%2Fconftest.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering#diff-YXVkYmFja2VuZC9jb3JlL2NvbmZ0ZXN0LnB5) | `100.0% <100.0%> (ø)` | |
frankenjoe commented 3 months ago

@hagenw ups, just realized you had not yet assigned the PR to me. Sorry, I hope you were finished.

hagenw commented 3 months ago

Yes, I was just waiting for the CI pipelines to finish. All fine.