fix(decision-worker): avoid missing decision task handling - #161
Conversation
Signed-off-by: Shijie Sheng <liouvetren@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests.
... and 1 file with indirect coverage changes 🚀 New features to boost your workflow:
|
Signed-off-by: Shijie Sheng <liouvetren@gmail.com>
Code Review ✅ Approved 1 resolved / 1 findingsSets return_new_decision_task to false in the decision worker and improves local integration testing, addressing the hardcoded log field finding. ✅ 1 resolved✅ Quality: Log field hardcodes return_new_decision_task=True after fix
OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
What changed?
Why?
return_new_decision_task flag controls whether responddecisioncompletetask would return a new decision. It could happen when
How did you test it?
Added async gather integration test to avoid regression.
Potential risks
Release notes
Documentation Changes