add support for /tl:[auto|on|off] msbuild flag

smoothdeveloper commented 6 months ago

Since this only works with dotnet sdk >= 8, the default case (specifying Auto) will not emit the tag.

TODO

Feel free to open the PR and ask for help

[ ] New (API-)documentation for new features exist (Note: API-docs are enough, additional docs are in help/markdown)
[x] unit or integration test exists (or short reasoning why it doesn't make sense)

Note: Consider using the CreateProcess API which can be tested more easily, see https://github.com/fsharp/FAKE/pull/2131/files#diff-4fb4a77e110fbbe8210205dfe022389b for an example (the changes in the DotNet.Testing.NUnit module)
[ ] boy scout rule: "leave the code behind in a better state than you found it" (fix warnings, obsolete members or code-style in the places you worked in)
[ ] (if new module) the module has been linked from the "side navigation" menu, edit docs/data.json.
[ ] (if new module) the module is in the correct namespace.
[ ] (if new module) the module is added to Fake.sln (dotnet sln Fake.sln add src/app/Fake.*/Fake.*.fsproj)
[ ] Fake API guideline is honored

smoothdeveloper commented 6 months ago

tried it (specificying TerminalLogger = On) in the build.fsx in this repository and it seems to work:

smoothdeveloper commented 6 months ago

now with <para/> tags.

xperiandri commented 6 months ago

Do you have any ideas why build fails?

smoothdeveloper commented 6 months ago

We should investigate but I doubt it is related to this change.

xperiandri commented 6 months ago

Probably something bad with an agent. Maybe update the version of image