Resolve CA1863 by using emitted methods

[JustArchi]

Checklist

• I read and understood ASF's Contributing guidelines
• I also read Setting-up and FAQ, I don't need help, this is an enhancement idea
• This is not a duplicate of an existing issue
• My idea doesn't duplicate existing ASF functionality described on the wiki
• I believe that my idea falls into ASF's scope and should be offered as part of ASF built-in functionality
• My idea doesn't violate the Steam Subscriber Agreement
• My idea doesn't violate the Steam Online Conduct
• This is not ASF-ui suggestion

Enhancement purpose

ASF ignored CA1863 for a while now due to dotnet/roslyn-analyzers#7032. We should aim to resolve it now that we have an option.

Solution

I did the initial part of using the new source generator in 7a8e209 - EmitFormatMethods="true" can be used for telling source generator to emit format options, afterwards, we need to do a lot of manual labour to replace every single occurance of string.Format() that we have today with the new method.

Why currently available solutions are not sufficient?

Existing string.Format() calls is a lot of boilerplate code, CA1863 is just a cherry on top. Now that there is a better solution to deal with this, we should prefer it.

Can you help us with this enhancement idea?

Yes, I can code the solution myself and send a pull request

Additional info

There are cases where plugins use ASF localization in part to avoid more work on their end - we'll need to #pragma warning disable those places before EmitFormatMethods="true" generates public methods.

dotnet/roslyn-analyzers#7360

[JustArchi]

For plugin creators there is only one visible breaking change, ASF no longer silences CA1863 warning by default: afa602f

If you need to silence it, you can consider adding it in your own plugin project: Directory.Build.props or .csproj:

	<PropertyGroup>
		<NoWarn>$(NoWarn),CA1863</NoWarn>
	</PropertyGroup>