-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Remove edition warning in cargo script w/o frontmatter #16598
Copy link
Copy link
Closed as not planned
Labels
C-feature-requestCategory: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`S-propose-closeStatus: A team member has nominated this for closing, pending further input from the teamStatus: A team member has nominated this for closing, pending further input from the teamZ-scriptNightly: cargo scriptNightly: cargo script
Metadata
Metadata
Assignees
Labels
C-feature-requestCategory: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`S-propose-closeStatus: A team member has nominated this for closing, pending further input from the teamStatus: A team member has nominated this for closing, pending further input from the teamZ-scriptNightly: cargo scriptNightly: cargo script
Type
Fields
Give feedbackNo fields configured for issues without a type.
View all comments
Problem
From #12207 (comment)
I don't think a warning should be printed on a script without any frontmatter.
Gives output:
Proposed Solution
Do not emit warning when no edition is specified for a cargo script. Defaulting to the latest edition is a perfectly reasonable default, and is what anyone would expect. Warnings on bare rust scripts without frontmatter are unnecessary noise for users who may not care about edition stability. If you care about stability, you can still specify it easily with frontmatter.
Notes
N/A