Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore: @requires_dependencies decorator for methods that use optional dependencies #118

Open
micpst opened this issue Oct 18, 2024 · 0 comments
Open

chore: @requires_dependencies decorator for methods that use optional dependencies #118

micpst opened this issue Oct 18, 2024 · 0 comments
Labels
chore Changes to the build process or repo setup

Comments

@micpst
Copy link
Collaborator

micpst commented Oct 18, 2024 •
edited
Loading

In #106 (comment) we introduced the @requires_dependencies decorator, which allows us to automatically throw an exception if we are missing some dependencies for a function. We currently use this in document sources, but we should apply it to all components that use optional dependencies.
@micpst micpst changed the title I'm testing a new way of handling optional dependencies. Since managing error messages from extra deps is starting to get tedious, I decided to wrap them in @requires_dependencies decorator that handles them automatically. chore: @requires_depenecies decorator for methods that use optional dependencies Oct 18, 2024
@micpst micpst added the chore Changes to the build process or repo setup label Oct 18, 2024
@micpst micpst changed the title chore: @requires_depenecies decorator for methods that use optional dependencies chore: @requires_dependencies decorator for methods that use optional dependencies Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
chore Changes to the build process or repo setup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@micpst