cargo icon indicating copy to clipboard operation
cargo copied to clipboard

Published 20 hours ago • verified publisher icon rust-lang

Strip cargo lints table

Open Muscraft opened this issue 2 months ago • 2 comments

[lints.cargo] should be removed when publishing/packaging to avoid unintentional MSRV bumps, and this PR makes it so the table is removed.

Muscraft avatar Apr 25 '24 20:04 Muscraft

r? @weihanglo

rustbot has assigned @weihanglo. They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

rustbot avatar Apr 25 '24 20:04 rustbot

I was wondering if we should just completely strip the table on publish.

This got me thinking, is there a downside to doing it completely? If so, is there a downside to doing it here?

Maybe if forbid is used but that won't work anyways.

If we wanted to include it, we could make it based on your package.rust-version.

epage avatar Apr 25 '24 21:04 epage

:umbrella: The latest upstream changes (presumably #13713) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Apr 29 '24 19:04 bors

#13833 supersedes this

Muscraft avatar May 01 '24 03:05 Muscraft