Avoid code coverage run on push trigger

[reyang]

I think we're not enforcing code coverage today? It is a bit misleading to see the latest push failed the CI:

[cijothomas]

@eddynaka do you recollect the reason behind adding this? We added to get code coverage comparison with main branch or something?

[eddynaka]

@eddynaka do you recollect the reason behind adding this? We added to get code coverage comparison with main branch or something?

Hi @cijothomas , i think the important part is that we have to run in main and in the branch. Because those would be used to compare between changes. So, the push itself isn't needed.

[codecov]

Codecov Report

Merging #1849 (a61b520) into main (cb066cb) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1849      +/-   ##
==========================================
- Coverage   83.77%   83.69%   -0.09%     
==========================================
  Files         187      187              
  Lines        5967     5967              
==========================================
- Hits         4999     4994       -5     
- Misses        968      973       +5     

Impacted Files	Coverage Δ
...ZPages/Implementation/ZPagesExporterEventSource.cs	56.25% <0.00%> (-6.25%)	⬇️
...emetry.Api/Internal/OpenTelemetryApiEventSource.cs	76.47% <0.00%> (-5.89%)	⬇️
...lemetry/Internal/SelfDiagnosticsConfigRefresher.cs	86.79% <0.00%> (-2.84%)	⬇️
...ter.ZPages/Implementation/ZPagesActivityTracker.cs	100.00% <0.00%> (+2.85%)	⬆️