-
Notifications
You must be signed in to change notification settings - Fork 717
Labels
coregood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededreleases-note/refactorThe PR does a refactor on code or has a title that begins with "refactor"The PR does a refactor on code or has a title that begins with "refactor"
Description
Currently, we have several spare parts handling jiff::Timestamp and related datetime logics.
Some of them are located at jiff_util.rs while others are inlined, so you can search jiff:: to find them.
To handle datetime logic modular and centralized, we can follow how reqsign encapsulates it:
- https://github.com/apache/opendal-reqsign/blob/60d1ca93/core/src/time.rs
- refactor: chrono to jiff opendal-reqsign#634
This should be easy to follow so I mark it as a good first issue. Feel free to take it over and submit a patch for review.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
coregood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededreleases-note/refactorThe PR does a refactor on code or has a title that begins with "refactor"The PR does a refactor on code or has a title that begins with "refactor"