Make segment deletion task idempotent#7380
Conversation
|
Someone is attempting to deploy a commit to the Flagsmith Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit a19b91d. Configure here.
a19b91d to
986e993
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7380 +/- ##
==========================================
- Coverage 98.44% 98.37% -0.07%
==========================================
Files 1398 1399 +1
Lines 52628 52522 -106
==========================================
- Hits 51808 51667 -141
- Misses 820 855 +35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
71cace6 to
34db9f7
Compare
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
34db9f7 to
e5cdfa5
Compare
e5cdfa5 to
755ba14
Compare

Summary
segments.tasks.delete_segmentignore already-deleted/missing segments by catchingSegment.DoesNotExistFixes #7354.
Testing
git diff --check origin/main..HEADpoetry run pytest tests/unit/segments/test_unit_segments_tasks.py -qpoetry run ruff format segments/tasks.py tests/unit/segments/test_unit_segments_tasks.py --checkpoetry run ruff check segments/tasks.py tests/unit/segments/test_unit_segments_tasks.py