-
Notifications
You must be signed in to change notification settings - Fork 53
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
chore(host): Refactor fetchers #995
Conversation
📚
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files☔ View full report in Codecov by Sentry. |
03615b9
to
b98b760
Compare
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.
I actually like the hint handler abstraction, think it works fairly clean. Can always improve later on
Overview
Refactors the fetchers for the
single
andinterop
host modes into a shared type, where hint handling can be customized via theHintHandler
trait.makes progress on #987, but doesn't yet implement support on the client side.