-
Notifications
You must be signed in to change notification settings - Fork 24
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
Buddy doesn't work on the C# target. #93
Comments
Hi, the C# target hasn't been well tested for the last couple of year(s), so I'm not surprised some issues has been introduced. I don't have too much time at the moment, so if you can give it a try and make a PR, that'll be much appreciated. If not I'll take a look when I get an opportunity. |
Fair enough! I word though appreciate it you could point me in a direction of where to look in the source. Just roughly, should I look north or south? |
Of course. :) The error seems related to should, and the should class is pretty much self-contained, so take a look and see if there are some casting issues there specifically for C#, maybe with the generics. |
Ok, took some time to dig into the problem and it seems to be related to ShouldInt(& ShouldFloat) class that have this construction:
If I change it to Btw. it seems that in some cases the ShouldFloat class is used when testing types of Int - haven't quite figured out why that happens. Is this a known issue? |
Running this test on the C# target:
fails with the following error:
The text was updated successfully, but these errors were encountered: