Remove net452 from shims project

[joaopgrassi]

Another project from #2138

• CHANGELOG.md updated for non-trivial changes
• Design discussion issue #
• Changes in public API reviewed

[reyang]

LGTM.

[cijothomas]

Please add net461 as there is no other netframework target here.

[codecov]

Codecov Report

Merging #2146 (4e6edfe) into main (a3867d4) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2146      +/-   ##
==========================================
- Coverage   85.00%   84.99%   -0.02%     
==========================================
  Files         187      187              
  Lines        6124     6124              
==========================================
- Hits         5206     5205       -1     
- Misses        918      919       +1     

Impacted Files	Coverage Δ
...Telemetry/Internal/SelfDiagnosticsEventListener.cs	95.34% <0.00%> (-0.78%)	⬇️

[joaopgrassi]

I think I fixed the remaining issues @cijothomas.

Just something I noticed: Is there a reason why the package Microsoft.CodeAnalysis.PublicApiAnalyzers is only added on Windows? I ran a build/test locally and didn't get any errors because I use Linux, but then I saw the CI failing..

In order to generate the public api files, I had to manually add the package and run dotnet format -a warn. Since all worked fine, I wonder if we can't just always add the package? Then people would not forget to add the api files like I did. It's the first time I use this, so just asking to understand it.

[cijothomas]

I think I fixed the remaining issues @cijothomas.

Just something I noticed: Is there a reason why the package Microsoft.CodeAnalysis.PublicApiAnalyzers is only added on Windows? I ran a build/test locally and didn't get any errors because I use Linux, but then I saw the CI failing..

In order to generate the public api files, I had to manually add the package and run dotnet format -a warn. Since all worked fine, I wonder if we can't just always add the package? Then people would not forget to add the api files like I did. It's the first time I use this, so just asking to understand it.

I don't recollect why... If you can make PR to do it in both win/linux, and if that makes its easy for linux based devs to catch issues easily, we can take it. Thanks!