Skip to content
This repository has been archived by the owner on Nov 7, 2024. It is now read-only.

Cargo.toml: Set resolver = 2 #520

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Conversation

cgwalters
Copy link
Member

@cgwalters cgwalters commented Aug 28, 2023

There's a warning in newer Rust 1.72 about this, we do want the v2
resolver.

@cgwalters cgwalters changed the title Buildsys tweaks Cargo.toml: Set resolver = 2 Aug 28, 2023
@cgwalters
Copy link
Member Author

oooo our own CI was actually relying on the old feature unification. Horray for CI though

@cgwalters cgwalters added the needs-rework/failing-ci Legitimate CI failure label Aug 28, 2023
There's a warning in newer Rust 1.72 about this, we do want the v2
resolver.

And adjust our build/CI configuration to explicitly enable
`internal-testing-api` when building our binary in CI.
Copy link
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

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

lgtm

@cgwalters cgwalters merged commit c5addba into ostreedev:main Aug 28, 2023
8 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants