Set PublishReferencesDocumentationFiles=false in fake-cli

[Numpsy]

Description

Just a thought when looking at the build output for fake-cli - Is there a need to have the XML documentation files for a bunch of the FAKE libraries included in the fake-cli NuGet package?

e.g.

[Thorium]

Maybe they help for API-docs (https://fake.build/reference/index.html) or user tool-tips?

[Numpsy]

The documentation XML files in the individual library nuget packages can be used for intellisense and such, and the doc comments themselves are used by the published documentation, but I didn't think the files in the dotnet global tool would be loaded as the package isn't attached to consuming projects like a library is (naybe I'm wrong there though)

[Thorium]

Many people still use Fake not via cli-tool but as "dotnet fsi build.fsx" with independent library dependencies, and I think xml-docs there will help.

[xperiandri]

@Numpsy Do you disable XML docs output only for CLI tool?

[Numpsy]

@Numpsy Do you disable XML docs output only for CLI tool?

That was the intent - e.g. in my local build the Fake.DotNet.Cli NuGet package contains the documentation XML file, but the fake-cli package doesn't -

[xperiandri]

@Numpsy let's merge?

[github-actions]

Test Results

    9 files  ±0      9 suites  ±0   37m 26s ⏱️ - 9m 41s
  439 tests ±0    439 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 251 runs  ±0  1 251 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 429cf51. ± Comparison against base commit 6f2fc43.