Skip to content

Conversation

@sergey-misuk-valor
Copy link
Contributor

@sergey-misuk-valor sergey-misuk-valor commented Dec 19, 2025

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 96.55172% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.33%. Comparing base (f0d9cef) to head (24148c7).
⚠️ Report is 18 commits behind head on develop.

Files with missing lines Patch % Lines
src/hope_dedup_engine/apps/api/celery_tasks.py 75.00% 1 Missing ⚠️
...ope_dedup_engine/apps/api/deduplication/process.py 85.71% 0 Missing and 1 partial ⚠️
src/hope_dedup_engine/apps/faces/celery_tasks.py 96.42% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #237      +/-   ##
===========================================
+ Coverage    96.31%   96.33%   +0.02%     
===========================================
  Files           82       82              
  Lines         1955     1993      +38     
  Branches       117      122       +5     
===========================================
+ Hits          1883     1920      +37     
- Misses          53       54       +1     
  Partials        19       19              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sergey-misuk-valor sergey-misuk-valor force-pushed the feature/273506-add_django_celery_boost branch from 0c3e2e7 to f3135f3 Compare December 29, 2025 07:44
)


@admin.register(DedupJob)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let create a base class for these Job classes. So we can have a single admin mage where to monitor all the tashs

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sergey-misuk-valor pls check this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@saxix is it good now?

@domdinicola domdinicola merged commit fb94810 into develop Jan 9, 2026
21 checks passed
@domdinicola domdinicola deleted the feature/273506-add_django_celery_boost branch January 9, 2026 14:19
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.

5 participants