Skip to content

Bugfix: Missing id on chain#8798

Merged
Nusnus merged 5 commits intocelery:mainfrom
Katz-Consulting-Group:blm-334
Jan 17, 2024
Merged

Bugfix: Missing id on chain#8798
Nusnus merged 5 commits intocelery:mainfrom
Katz-Consulting-Group:blm-334

Conversation

@Nusnus
Copy link
Member

@Nusnus Nusnus commented Jan 17, 2024

Fixes: #8786, #8782

Of course all of the new tests are failing without the fix and passing with the fix.

@Nusnus Nusnus added this to the 5.4 milestone Jan 17, 2024
@Nusnus Nusnus requested a review from thedrow January 17, 2024 13:19
@Nusnus Nusnus self-assigned this Jan 17, 2024
@Nusnus Nusnus changed the title Blm 334 Bugfix: Missing id on chain Jan 17, 2024
@Nusnus Nusnus marked this pull request as ready for review January 17, 2024 13:20
@codecov
Copy link

codecov bot commented Jan 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.18%. Comparing base (78c06af) to head (a72049e).
Report is 396 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8798   +/-   ##
=======================================
  Coverage   81.17%   81.18%           
=======================================
  Files         148      148           
  Lines       18521    18522    +1     
  Branches     3165     3165           
=======================================
+ Hits        15035    15037    +2     
  Misses       3197     3197           
+ Partials      289      288    -1     
Flag Coverage Δ
unittests 81.16% <100.00%> (+<0.01%) ⬆️

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.

@Nusnus Nusnus merged commit ad49065 into celery:main Jan 17, 2024
@Nusnus Nusnus deleted the blm-334 branch January 17, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Unexpected Behaviors for a Chain of Groups, when Groups contain failing tasks

2 participants