-
Notifications
You must be signed in to change notification settings - Fork 772
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
Port a test from Paket over to FSharp.Core tests #3350
Conversation
ok it seems to work on master. |
Mhm it also seems to work against the 4.2.1 branch. I'm lost |
Correspondign Paket branch: fsprojects/Paket#2553 |
Ok it clearly hangs in paket when we update fsharp.core. |
Ok so it seems that it's not test what's hanging. I commented that test out from paket's test suite and it's still hanging. So I still believe the new fsharp.core is flawed, but not with this test here. @matthid and myself will investigate further. Anyway @KevinRansom please feel free to merge this test here. It's good to have it in the code base. |
@forki |
* add another test for #3254, ported from paket. related to #3350 and fsprojects/Paket#2553 * Fix exception message
This test was added to Paket by @Thorium in August 2016 and worked up to FSharp.Core 4.1.17 but hangs in 4.2.1.
/cc @eiriktsarpalis @dsyme @matthid @Thorium