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.

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

Loader: Fix missing filepath_from_context classmethod decorator #86

Conversation

BigRoy
Copy link
Contributor

@BigRoy BigRoy commented Sep 4, 2024

Changelog Description

Fix missing filepath_from_context classmethod decorator that it should match from the parent class.

Additional info

My bad in this PR: #26 - I missed it there.

Testing notes:

  1. Validate loaders work.
  2. Or just compare it against the parent class.

@BigRoy BigRoy added the type: bug Something isn't working label Sep 4, 2024
@BigRoy BigRoy self-assigned this Sep 4, 2024
@BigRoy BigRoy merged commit dc72143 into ynput:develop Sep 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants