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

don't set MSRV and only run doctests on nightly #37

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Conversation

jswrenn
Copy link
Collaborator

@jswrenn jswrenn commented Feb 6, 2024

We have too many dependencies to actually guarantee our MSRV.

We only run doctests on nightly so we don't have to generalize doctests for slightly different line numbers across different Rust versions. Our integration tests do generalize in this way, so we still run those on both stable and nightly.

@jswrenn jswrenn force-pushed the abandon-msrv branch 2 times, most recently from 7960ce2 to 31005fa Compare February 6, 2024 16:15
We have too many dependencies to actually guarantee our MSRV.

We only run doctests on nightly so we don't have to generalize
doctests for slightly different line numbers across different Rust
versions. Our integration tests do generalize in this way, so we
still run those on both stable and nightly.
@jswrenn jswrenn merged commit ef988a0 into main Feb 6, 2024
8 checks passed
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.

1 participant