Skip to content

Add missing HasLink instance for DeepQuery (#1784) #1814

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kutyel
Copy link
Contributor

@kutyel kutyel commented Apr 7, 2025

This solution is based in @saurabhnanda's proposed solution, let me know if it needs further improving @clementd-fretlink @theophile-fl 🙏🏻 👀

Fixes #1784

@tchoutri
Copy link
Contributor

tchoutri commented Apr 8, 2025

@kutyel Please rebase on HEAD, it seems to be fixed. :)

@kutyel kutyel force-pushed the 1784-haslink-instance-for-deepquery branch from 15e5d18 to 177a720 Compare April 8, 2025 09:39
@kutyel kutyel force-pushed the 1784-haslink-instance-for-deepquery branch 2 times, most recently from 83f138c to 594fa99 Compare April 9, 2025 10:05
@kutyel
Copy link
Contributor Author

kutyel commented Apr 10, 2025

@tchoutri @clementd-fretlink build finally passed, feel free to review when you have time please 🙏🏻 🙌🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Is DeepQuery missing a HasLink instance?
2 participants