-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add repositories argument to audb.available() #489
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
db9e6eb
to
8d46aaa
Compare
@sourcery-ai review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @hagenw - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟡 Testing: 1 issue found
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the issues that I had
(typing related and a more explicit docstring) are settled.
Proceeding with approval.
Add
repositories
argument toaudb.available()
to be able to limit the repositories, in which it should search. Before, a user needed to modifyaudb.confg.REPOSITORIES
before callingaudb.available()
to limit its output to certain repositories.Summary by Sourcery
Add a 'repositories' argument to the audb.available() function to enable users to limit the search to specified repositories, enhancing the function's flexibility and usability.
New Features:
Tests: