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

Having two infer do not work #169

Closed
magnetik opened this issue Jun 30, 2022 · 3 comments
Closed

Having two infer do not work #169

magnetik opened this issue Jun 30, 2022 · 3 comments

Comments

@magnetik
Copy link
Contributor

Hi,

It seems that having two infer registered do not work.

I've done a "as simple as possible" repro code here : https://github.com/magnetik/valinor-repro

When I register A before B I have an error on B, and when I register B before A, I have an error on A.

Is there something I'm missing?

@romm
Copy link
Member

romm commented Jul 1, 2022

Hi @magnetik, good catch! Thanks for the repo, helped a lot to reproduce the issue locally.

Could you check #170 to see if that fixes your issue?

@magnetik
Copy link
Contributor Author

magnetik commented Jul 4, 2022

I just tested and it works (the Root class was not correct in my example, but it works when fixed) !

@romm
Copy link
Member

romm commented Jul 4, 2022

Perfect, thank you!

@romm romm closed this as completed Jul 4, 2022
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

No branches or pull requests

2 participants