Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): change precision point of time #4401

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

dracarys18
Copy link
Member

Type of Change

  • Bugfix

Description

Changed timeprecision from 3 to 6

Motivation and Context

This fixes the bug in time crate where the seconds get rounded off to 60. More on this issue here

How did you test it?

Cannot be tested on sandbox.

You can clone this repo and do,

cargo r | tee test.txt
cat test.txt | rg ':60.'

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code

@dracarys18 dracarys18 added A-core Area: Core flows S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Apr 19, 2024
@dracarys18 dracarys18 added this to the April 2024 milestone Apr 19, 2024
@dracarys18 dracarys18 self-assigned this Apr 19, 2024
@dracarys18 dracarys18 requested a review from a team as a code owner April 19, 2024 13:14
SanchithHegde
SanchithHegde previously approved these changes Apr 19, 2024
@dracarys18 dracarys18 changed the title fix: change time precision to 6 fix(custom_serde): change the format config to default one Apr 19, 2024
akshay-97
akshay-97 previously approved these changes Apr 19, 2024
@dracarys18 dracarys18 dismissed stale reviews from akshay-97 and SanchithHegde via dfd6b30 April 22, 2024 08:27
@dracarys18 dracarys18 changed the title fix(custom_serde): change the format config to default one chore(deps): change dependency to time fork Apr 22, 2024
@dracarys18 dracarys18 changed the title chore(deps): change dependency to time fork chore(deps): change precision point of time Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-core Area: Core flows S-waiting-on-review Status: This PR has been implemented and needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants