Skip to content

fix(script): surpress unused_features lint for embedded#16714

Merged
epage merged 1 commit intorust-lang:masterfrom
weihanglo:script
Mar 6, 2026
Merged

fix(script): surpress unused_features lint for embedded#16714
epage merged 1 commit intorust-lang:masterfrom
weihanglo:script

Conversation

@weihanglo
Copy link
Member

@weihanglo weihanglo commented Mar 6, 2026

What does this PR try to resolve?

rust-lang/rust#152164 added a warn-by-default unused_features lint.
Cargo injects #![feature(frontmatter)] for all embedded scripts,
but scripts without frontmatter syntax never trigger feature gate check,
and causes causing the lint warning.

Given the stabilization FCP of frontmatter is complete already,
we could expect it will soon be stabilized (?).

It should be fine we suppress this new unstable lint.

How to test and review this PR?

Test suite passes.
See #16713 (comment) for more.

rust-lang/rust#152164 added a warn-by-default `unused_features` lint.
Cargo injects `#![feature(frontmatter)]` for all embedded scripts,
but scripts without frontmatter syntax never trigger feature gate check,
and causes causing the lint warning.

Given the stabilization FCP of frontmatter is complete already,
we could expect it will soon be stabilized (?).

It should be finewe suppress this new unstable lint.
@weihanglo weihanglo added the Z-script Nightly: cargo script label Mar 6, 2026
@rustbot rustbot added A-build-execution Area: anything dealing with executing the compiler S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 6, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 6, 2026

r? @epage

rustbot has assigned @epage.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from ehuss, epage

@epage epage added this pull request to the merge queue Mar 6, 2026
Merged via the queue into rust-lang:master with commit 01e42b9 Mar 6, 2026
29 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 6, 2026
@weihanglo weihanglo deleted the script branch March 6, 2026 13:52
@ehuss ehuss added the beta-nominated Nominated to backport to the beta branch. label Mar 6, 2026
ehuss pushed a commit to ehuss/cargo that referenced this pull request Mar 6, 2026
…16714)

### What does this PR try to resolve?

rust-lang/rust#152164 added a warn-by-default
`unused_features` lint.
Cargo injects  `#![feature(frontmatter)]` for all embedded scripts,
but scripts without frontmatter syntax never trigger feature gate check,
and causes causing the lint warning.

Given the stabilization FCP of frontmatter [is
complete](rust-lang/rust#148051 (comment))
already,
we could expect it will soon be stabilized (?).

It should be fine we suppress this new unstable lint.

### How to test and review this PR?

Test suite passes.
See
<rust-lang#16713 (comment)>
for more.
weihanglo added a commit that referenced this pull request Mar 6, 2026
This backports the following to fix the beta branch and to reduce
spurious errors:

- #16700 -- test(replace): Mark a
test as non-deterministic
- #16706 -- test(git): Mark a
test as non-deterministic
- #16714 -- fix(script): surpress
unused_features lint for embedded
rust-bors bot pushed a commit to rust-lang/rust that referenced this pull request Mar 13, 2026
Update cargo submodule

21 commits in 90ed291a50efc459e0c380d7b455777ed41c6799..cbb9bb8bd0fb272b1be0d63a010701ecb3d1d6d3
2026-03-05 15:11:25 +0000 to 2026-03-13 14:34:16 +0000
- fix(git): preserve SCP-like submodule URLs for fetch (rust-lang/cargo#16744)
- Fix 16732 (rust-lang/cargo#16746)
- fix(rustdoc): switch to new `--emit` options (rust-lang/cargo#16739)
- fix(git): convert SCP-like submodule URLs to ssh:// format (rust-lang/cargo#16727)
- fix(core): typo (rust-lang/cargo#16738)
- CARGO_TARGET_DIR doesn't have to be relative (rust-lang/cargo#16735)
- chore(ci): Detect user changes to src/etc/man (rust-lang/cargo#16736)
- util: Exclude from iCloud Drive sync on macOS (rust-lang/cargo#16728)
- fix(compile): Stop on denying warnings without --keep-going  (rust-lang/cargo#16725)
- feat(shell): Support OSC 9;4 progress on ptyxis (rust-lang/cargo#16730)
- Let git decide when to run gc (rust-lang/cargo#16459)
- chore(deps): update cargo-semver-checks to v0.47.0 (rust-lang/cargo#16723)
- fix(compile): Turn warning summaries into errors also (rust-lang/cargo#16721)
- fix(fix): Switch from ad-hoc to structured warnings (rust-lang/cargo#16711)
- fix: ignore implicit std dependencies in `unused-crate-dependencies` lint (rust-lang/cargo#16677)
- chore(deps): update msrv (3 versions) to v1.92 (rust-lang/cargo#16716)
- chore(deps): update msrv (1 version) to v1.94 (rust-lang/cargo#16710)
- fix(script): surpress `unused_features` lint for embedded (rust-lang/cargo#16714)
- Split `build-dir` lock into dedicated lock  (rust-lang/cargo#16708)
- Add missing truncate when writing .crate files (rust-lang/cargo#16688)
- refactor(fix): Prep for annotate-snippets (rust-lang/cargo#16702)
@rustbot rustbot added this to the 1.96.0 milestone Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-build-execution Area: anything dealing with executing the compiler beta-nominated Nominated to backport to the beta branch. Z-script Nightly: cargo script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants