Skip to content

fix(sqlite): avoid deprecated time format components - #62

Merged
lovasoa merged 1 commit into
mainfrom
codex/fix-time-format-deprecations
Sep 27, 2026
Merged

lovasoa merged 1 commit into
mainfrom
codex/fix-time-format-deprecations

Conversation

@lovasoa

@lovasoa lovasoa commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator

The committed time 0.3.55 dependency deprecates the manually constructed date/time components and byte literals, causing 40 errors under CI's -D warnings even with Rust pinned to 1.98.1. This is the failure blocking #61.

Build the existing SQLite format descriptions with time's compile-time macro, retaining the same optional separators, seconds, fractions and offsets. This avoids a dependency bump or warning suppression.

Validation on Rust 1.98.1:

  • cargo fmt --all -- --check
  • Core Clippy with all database/type features and -D warnings, for both native-tls and rustls.
  • Root Clippy with macros and all-targets Clippy using the existing native-tls CI feature sets.
  • All four existing SQLite time type integration tests passed (date, time, primitive datetime and offset datetime).

Local test/build commands used --locked; commands involving vendored OpenSSL used OPENSSL_NO_VENDOR=1 to use the installed system OpenSSL.

@lovasoa
lovasoa merged commit 3458cd8 into main Sep 27, 2026
27 checks passed
@lovasoa
lovasoa deleted the codex/fix-time-format-deprecations branch September 27, 2026 23:01
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.

1 participant