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

Use std duration v0.4.x #860

Closed
wants to merge 3 commits into from
Closed

Use std duration v0.4.x #860

wants to merge 3 commits into from

Conversation

esheppa
Copy link
Collaborator

@esheppa esheppa commented Nov 2, 2022

No description provided.

@esheppa
Copy link
Collaborator Author

esheppa commented Nov 2, 2022

still need to apply most of 74b087f onto this before its ready for any review

@esheppa esheppa force-pushed the use-std-duration-v0.4.x branch 2 times, most recently from 61277d5 to 71416a5 Compare November 12, 2022 12:20
@esheppa
Copy link
Collaborator Author

esheppa commented Nov 12, 2022

Removed TimeDelta as per discussion in #858 (comment)

@djc djc marked this pull request as ready for review November 13, 2022 07:17
@djc
Copy link
Contributor

djc commented Nov 13, 2022

Huh, not sure how I did that but it wasn't on purpose.

@esheppa
Copy link
Collaborator Author

esheppa commented Nov 23, 2022

Would this be better as three or four seperate PR's? One each for Naive Date, Time, DateTime and potentially DateTime? (I think it may be able to be separated out now that we aren't removing the old code

@djc
Copy link
Contributor

djc commented Nov 23, 2022

Probably easier to digest that way!

@esheppa
Copy link
Collaborator Author

esheppa commented Nov 24, 2022

Closing in favour of #895 (and future PRs for NaiveDate updates and (Naive)DateTime updates)

@esheppa esheppa closed this Nov 24, 2022
@pitdicker pitdicker deleted the use-std-duration-v0.4.x branch April 9, 2024 14:00
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.

None yet

2 participants