dotnet / roslyn-analyzers

MIT License
1.58k stars 465 forks source link

Remove CodeQL tasks duplicating injected ones #7267

Closed jjonescz closed 6 months ago

jjonescz commented 6 months ago

Follow up on https://github.com/dotnet/roslyn-analyzers/pull/7253.

Currently official builds are getting stuck on CodeQL step: https://dnceng.visualstudio.com/internal/_build/results?buildId=2414766&view=results (interestingly, test run of https://github.com/dotnet/roslyn-analyzers/pull/7253 didn't get stuck, not sure why) Test run of an official build of this PR: https://dnceng.visualstudio.com/internal/_build/results?buildId=2414884&view=results This is like a change done in roslyn-sdk to resolve the same problem: https://github.com/dotnet/roslyn-sdk/pull/1152/commits/816cd02d2f99bde68614c6d833291b1a1fbbdff3

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.48%. Comparing base (00071ec) to head (2522861).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7267 +/- ## ========================================== - Coverage 96.48% 96.48% -0.01% ========================================== Files 1442 1442 Lines 345212 345212 Branches 11345 11345 ========================================== - Hits 333086 333073 -13 - Misses 9252 9263 +11 - Partials 2874 2876 +2 ```