Skip to content

types: add support for Jiff's Timestamp#1676

Merged
gwenn merged 2 commits intorusqlite:masterfrom
BurntSushi:ag/more-jiff
Apr 17, 2025
Merged

types: add support for Jiff's Timestamp#1676
gwenn merged 2 commits intorusqlite:masterfrom
BurntSushi:ag/more-jiff

Conversation

@BurntSushi
Copy link
Contributor

This is probably the most important type to have rusqlite traits
implemented for from Jiff.

Jiff's default parsing/formatting is already generally compatible with
what SQLite itself supports, so we start there.

This is probably the most important type to have rusqlite traits
implemented for from Jiff.

Jiff's default parsing/formatting is already generally compatible with
what SQLite itself supports, so we start there.
@codecov
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.93%. Comparing base (7228c9a) to head (1c3f8f1).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1676      +/-   ##
==========================================
+ Coverage   86.86%   86.93%   +0.06%     
==========================================
  Files          58       58              
  Lines       11088    11141      +53     
==========================================
+ Hits         9632     9685      +53     
  Misses       1456     1456              

☔ 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.

@gwenn
Copy link
Collaborator

gwenn commented Apr 17, 2025

Thanks !
Would you mind fixing the Clippy warning ?

@gwenn gwenn merged commit 0024ecf into rusqlite:master Apr 17, 2025
16 checks passed
@BurntSushi BurntSushi deleted the ag/more-jiff branch April 17, 2025 14:36
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.

2 participants