Skip to content

Conversation

@Himmelschmidt
Copy link
Contributor

@Himmelschmidt Himmelschmidt commented Dec 26, 2025

Use DateTime/DateTimeExt from apalis-sql. Includes migration to convert DATETIME → TIMESTAMP columns for UTC handling.

Related: apalis-dev/apalis#655

@codecov-commenter
Copy link

codecov-commenter commented Jan 7, 2026

Codecov Report

❌ Patch coverage is 36.36364% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/queries/list_workers.rs 0.00% 11 Missing ⚠️
src/from_row.rs 83.33% 1 Missing ⚠️
src/lib.rs 0.00% 1 Missing ⚠️
src/shared.rs 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Replace direct chrono dependency with DateTime/DateTimeExt from apalis-sql,
enabling support for both chrono and time datetime libraries via feature flags.

Changes:
- Add `chrono` (default) and `time` feature flags
- Use DateTimeExt::from_unix_timestamp instead of chrono::Utc.timestamp_opt
- Update apalis dependencies to 0.7.0-rc.2
- Add migration to convert datetime columns to TIMESTAMP
- Move chrono to dev-dependencies (only needed for tests/examples)

Related: apalis-dev/apalis#655
@Himmelschmidt Himmelschmidt force-pushed the feat/apalis-sql-datetime-abstraction branch from d1edfee to d6b9e23 Compare January 8, 2026 19:37
@geofmureithi geofmureithi merged commit 5e5d927 into apalis-dev:main Jan 9, 2026
12 checks passed
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.

3 participants