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

Add missing edition = "2021" to compile and runtime crates #21

Merged
merged 1 commit into from Oct 25, 2022

Conversation

MarijnS95
Copy link
Contributor

@MarijnS95 MarijnS95 commented Oct 20, 2022

Commit ad85f13 ("Seems like edition in workspace isn't applied? Update all to 2021") found that workspace.edition isn't a valid key and added it to most crates' individual [package] table, but didn't remove the key from [workspace] nor added it to the compile and runtime crates:

warning: ispc-rs/Cargo.toml: unused manifest key: workspace.edition

Note that [workspace.package] edition = "2021" exists since Rust 1.64, but still requires [package] edition.workspace = true to be repeated in every individual Cargo.toml for it to be inherited: https://doc.rust-lang.org/cargo/reference/workspaces.html#the-package-table

Commit ad85f13 ("Seems like edition in workspace isn't applied? Update
all to 2021") found that `workspace.edition` isn't a valid key and added
it to most crates' individual `[package]` table, but didn't remove the
key from `[workspace]` nor added it to the `compile` and `runtime`
crates.

Note that `[workspace.package] edition = "2021"` exists since Rust 1.64,
but still requires `[package] edition.workspace = true` to be repeated
in every individual `Cargo.toml` for it to be inherited:
https://doc.rust-lang.org/cargo/reference/workspaces.html#the-package-table
@Twinklebear
Copy link
Owner

Thanks for the fix @MarijnS95 !

@Twinklebear Twinklebear merged commit 39e0598 into Twinklebear:master Oct 25, 2022
@MarijnS95 MarijnS95 deleted the missing-edition branch October 25, 2022 12:03
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.

None yet

2 participants