Skip to content

fix(deps): correct minium chrono version and MSRV #324

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

Merged
merged 3 commits into from
Apr 9, 2021

Conversation

flub
Copy link
Contributor

@flub flub commented Apr 9, 2021

The sentry-core crate uses the chrono::DurationRound trait which was
only introduced in chrono 0.4.13. The sentry-types crate is updated
just to keep the versions in sync.

Floris Bruynooghe added 2 commits April 9, 2021 09:11
The sentry-core crate uses the chrono::DurationRound trait which was
only introduced in chrono 0.4.13.  The sentry-types crate is updated
just to keep the versions in sync.
The socket2 dependency (pulled in via hyper) uses this.
@flub flub changed the title fix(deps): correct miniump chrono version fix(deps): correct minium chrono version Apr 9, 2021
@flub
Copy link
Contributor Author

flub commented Apr 9, 2021

Do not be confused by the unfinished CI checks, everything works if we're ok with bumping the MSRV to 1.46 (I couldn't find this in any documentation so didn't update it). The branch protection needs to be updated if we approve this.

So this is as ready for review as it gets.

@flub flub changed the title fix(deps): correct minium chrono version fix(deps): correct minium chrono version and MSRV Apr 9, 2021
@Swatinem Swatinem merged commit 6abf8f1 into master Apr 9, 2021
@Swatinem Swatinem deleted the fix/chrono-version branch April 9, 2021 09:48
@Swatinem
Copy link
Member

Swatinem commented Apr 9, 2021

#316 would require a MSRV bump as well since actix bumped it as well.

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.

2 participants