Included existing NoWarn value

[nojaf]

I have an extra value NoWarn value in my Directory.Build.props.user.
This would allow it to come through in src/FSharp.Build/FSharp.Build.fsproj.

PS: When do you use <NoWarn>$(NoWarn);75</NoWarn> versus <NoWarn>FS0075</NoWarn>?

[psfinaki]

I think <NoWarn>$(NoWarn);(all the other warnings)</NoWarn> is the right approach.

[nojaf]

@psfinaki I was more referring to the 75 versus FS0075? Do both mean the same thing?

[psfinaki]

Hmm I suspect MSBuild would default a random number to C# compiler warnings. I cannot find explicit docs on that, although these explanations kind of imply that. Either way, FS0075 is more safe and explicit.

[baronfel]

@psfinaki it's not an MSBuild behavior, rather Roslyn chooses to interpret pure numbers as having the CS prefix. This is not considered good practice anymore in a world of many different components providing diagnostics (analyzers, linker, nuget, etc) and so we should always fully qualify these days.