tisonkun opened a new pull request, #643: URL: https://github.com/apache/opendal-reqsign/pull/643
We are now tightly coupled with `jiff`. I don't see we can encapsulate jiff entirely. Perhaps either: * `pub extern crate jiff as jiff02` and add a `jiff02` feature flag * Hide jiff Timestamp behind `pub struct reqsign_core::Timestamp(jiff:Timestamp)`, and expose methods as needed, like adding a duration, parsing/print specific timestamp format, etc. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@opendal.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
