Skip to content

Conversation

@Swatinem
Copy link
Contributor

(migrated from codecov/worker#1165)

This does what the title says, removing the task code related to the obsolete labelanalysis and staticanalysis, which were parts of ATS.

@Swatinem Swatinem self-assigned this Apr 28, 2025
@Swatinem Swatinem force-pushed the worker/swatinem/rm-analysistasks branch 2 times, most recently from 22a66ea to 1e55e0d Compare April 28, 2025 09:52
@codecov-notifications
Copy link

codecov-notifications bot commented Apr 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Apr 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.11%. Comparing base (1b59fc3) to head (d3508d9).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   94.24%   94.11%   -0.13%     
==========================================
  Files        1209     1204       -5     
  Lines       45539    45101     -438     
  Branches     1449     1449              
==========================================
- Hits        42916    42445     -471     
- Misses       2317     2350      +33     
  Partials      306      306              
Flag Coverage Δ
apiunit 96.36% <ø> (-0.02%) ⬇️
shared-docker-uploader 88.82% <ø> (-0.01%) ⬇️
workerintegration 61.71% <ø> (+0.53%) ⬆️
workerunit 90.91% <ø> (-0.44%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codspeed-hq
Copy link

codspeed-hq bot commented Apr 28, 2025

CodSpeed Performance Report

Merging #61 will not alter performance

Comparing worker/swatinem/rm-analysistasks (d3508d9) with main (b618863)

Summary

✅ 9 untouched benchmarks

@Swatinem Swatinem force-pushed the worker/swatinem/rm-analysistasks branch from 1e55e0d to eeee1bb Compare May 2, 2025 11:39
@seer-by-sentry
Copy link
Contributor

seer-by-sentry bot commented May 2, 2025

✅ Sentry found no issues in your recent changes ✅

@Swatinem Swatinem enabled auto-merge May 2, 2025 11:41
This does what the title says, removing the task code related to the obsolete labelanalysis and staticanalysis, which were parts of ATS.
@Swatinem Swatinem force-pushed the worker/swatinem/rm-analysistasks branch from eeee1bb to d3508d9 Compare May 2, 2025 11:49
@Swatinem Swatinem added this pull request to the merge queue May 2, 2025
Merged via the queue into main with commit a2a7e86 May 2, 2025
70 of 72 checks passed
@Swatinem Swatinem deleted the worker/swatinem/rm-analysistasks branch May 2, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants