From db7ce14ad7ca4f52eb26e40675948b54e4d31533 Mon Sep 17 00:00:00 2001 From: "Andres G. Aragoneses" Date: Fri, 18 Aug 2023 20:18:05 +0800 Subject: [PATCH] fsxc: default to warnings as errors Fixes https://github.com/nblockchain/fsx/issues/32 --- ReadMe.md | 2 +- fsxc/Fsxc.fs | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/ReadMe.md b/ReadMe.md index 9828c7a6..d705a494 100644 --- a/ReadMe.md +++ b/ReadMe.md @@ -7,7 +7,7 @@ THIS REPO IS FACING A COMPLETE OVERHAUL/REVAMP/RENOVATION IN ORDER TO SUPPORT .N Unfinished tasks so far: * Revamp this ReadMe.md file to remove any mentions to Mono or the legacy .NET4.x framework. * Remove legacy framework support (so that build system can converge into .fsx files instead of autotools in Unix + fsx in Windows). -* Make fsxc always enable warnAsError and fsx always disable it. +* Allow fsxc && fsx disable warnAsError (via --w flag? or --ignore-warnings). * After doing the above, make both fsx and fsxc always enable warnAsError for the FS0020 warning described in https://stackoverflow.com/questions/38202685/fsx-script-ignoring-a-function-call-when-i-add-a-parameter-to-it * Try creating VMs for CI that uninstall .NETCore/.NET6 completely (not just the dotnet executable removal hack), to make sure legacy framework build still works there. * Try creating VMs for CI that uninstall Mono/.NET4.x completey (e.g. for macOS see: https://github.com/mono/website/commit/490797429d4b92584394292ff69fbdc0eb002948 ) diff --git a/fsxc/Fsxc.fs b/fsxc/Fsxc.fs index cc6cfc90..e09383fc 100644 --- a/fsxc/Fsxc.fs +++ b/fsxc/Fsxc.fs @@ -710,6 +710,9 @@ let fsi = { CommandLineArgs = System.Environment.GetCommandLineArgs() } false + + true + """ let initialProjectContents = initialProjectContents