Skip to content

Conversation

crepererum
Copy link
Collaborator

Looking at IOx, we currently have 3 time-like dependencies:

  • time 0.1: this is a backwards-compat dependencies of chrono 0.4 and
    will be removed in chrono 0.5
  • time 0.3: only used by rskafka
  • chrono 0.4: used by everything else

Let's remove time 0.3 from that list. Also it seems that chrono gains
more traction recently, so IMHO it's the safer bet.

Also see #144.

@crepererum crepererum requested a review from domodwyer September 6, 2022 16:18
Looking at IOx, we currently have 3 time-like dependencies:

- time 0.1: this is a backwards-compat dependencies of chrono 0.4 and
  will be removed in chrono 0.5
- time 0.3: only used by rskafka
- chrono 0.4: used by everything else

Let's remove time 0.3 from that list. Also it seems that chrono gains
more traction recently, so IMHO it's the safer bet.

Also see #144.
Copy link
Contributor

@domodwyer domodwyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@crepererum crepererum added the automerge Instruct kodiak to merge the PR label Sep 7, 2022
@kodiakhq kodiakhq bot merged commit 3208e47 into main Sep 7, 2022
@kodiakhq kodiakhq bot deleted the crepererum/chrono branch September 7, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Instruct kodiak to merge the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants