-
Notifications
You must be signed in to change notification settings - Fork 104
Use F#10 for the main lib #614
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
base: master
Are you sure you want to change the base?
Conversation
475a5ed to
49c70bb
Compare
10.0.x 9.0.x
This reverts commit 714fe0b.
e213a85 to
ce22ec7
Compare
e213a85 to
654851c
Compare
654851c to
809178a
Compare
| <DefineConstants Condition=" '$(Configuration)' == 'Test'">$(DefineConstants);TEST_TRACE</DefineConstants> | ||
| <DefineConstants Condition=" '$(Configuration)' == 'Fable'">$(DefineConstants);FABLE_COMPILER</DefineConstants> | ||
| <DefineConstants Condition=" '$(Configuration)' == 'Fable4'">$(DefineConstants);FABLE_COMPILER;FABLE_COMPILER_4</DefineConstants> | ||
| <TargetFramework>net8.0</TargetFramework> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to use target framework net10?
|
@wallymathieu , @gusty : As part of fixing compiler regressions spotted in this PR, I was automatically running all targets and when successfully running them, I realized it also made an unintended push to gh-pages. Would you mind having a look and reverting if needed? |
No worries, I just reverted. Thanks ! |
With the introduction of F#9 there seems to be a breaking change in backwards compatibility.
Although we can't fix the compiler, we can produce a lib that will work when consumed from F#9, still targeting .net8