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

Pin rust version to 1.65 instead of pinning time-core #426

Closed

Conversation

timsueberkrueb
Copy link
Contributor

Summary

See #421 (comment)

Checklist

arlyon
arlyon previously requested changes Aug 18, 2023
Cargo.toml Outdated Show resolved Hide resolved
@arlyon
Copy link
Owner

arlyon commented Aug 19, 2023

looks like time now needs 1.67.0 so lets just bump our msrv to that while we are at it. change the version in cargo.toml and in the .github worflows (for CI)

@timsueberkrueb
Copy link
Contributor Author

@arlyon done, thank you!

@arlyon
Copy link
Owner

arlyon commented Aug 21, 2023

Running CI again, TY

@timsueberkrueb
Copy link
Contributor Author

@arlyon the CI failing is an unrelated issue I assume?

@timsueberkrueb
Copy link
Contributor Author

Hey @arlyon, any update on this one? Thanks :)

@timsueberkrueb
Copy link
Contributor Author

Hey @arlyon, it would be really cool if we could land this and have a new crates.io release containing the fix. Thanks!

@arlyon
Copy link
Owner

arlyon commented Sep 5, 2023

Hi there, tests aren't passing and it seems that rustc is segfaulting when used with cargo-llvm-cov. I am going to rip it out, because it just adds another layer of mess.

@arlyon
Copy link
Owner

arlyon commented Sep 5, 2023

Fixed in this PR #433

New release to crates coming now

@arlyon arlyon closed this Sep 5, 2023
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