-
Notifications
You must be signed in to change notification settings - Fork 56
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
Update TypeShape 8.0.1 -> 9.0.0 #226
Comments
That is really weird. Updating is certainly worth a shot, it won't hurt anything. However, TypeShape isn't pinned to a major version, are you sure you're not already deploying your application with v9? |
You are right @kerams it looks like it this is just paket lock from the SAFE template. Maybe that's the problem |
eiriktsarpalis/TypeShape@a8be8c7#diff-6a7780a357a197c700a5c9acdacd22efa6ea59e35cfb542b4ad370acf8766565R147-R151 In the meantime, I'm sure locking into v8 will help you. |
Yeah I was talking to @isaacabraham about it (he reported the issue) and I think we should either update TypeShape to 9.0.0 or set better version constraints to say TypeShape should be >= 8.0.1 && < 9.0.0 because now paket (correctly) chose the highest matching version of TypeShape because we have only >= 8.0.1 |
@Zaid-Ajaj could you share the full stacktrace? |
Are you consuming TypeShape using the nuget package or by embedding its source file? The former only targets netstandard2.0 which uses the workaround referenced in eiriktsarpalis/TypeShape@a8be8c7#diff-6a7780a357a197c700a5c9acdacd22efa6ea59e35cfb542b4ad370acf8766565R147-R151 |
@eiriktsarpalis The stack trace was posted in TypeShape/#45. Do you recommend to use the source file or the nuget package v9? |
Some weird FATAL runtime error occurs with TypeShape and it happens only on deployed application in azure (assuming it is running net5.0) but works locally just fine using either -c Debug or -c Release
So I am thinking it might have to do with TypeShape using an old reflection API so I think we should update it to latest 9.0.0 just in case. What do you think, @kerams?
The text was updated successfully, but these errors were encountered: