Skip to content

error when a command line flag lowers a -F forbid - #165669

Open
Albab-Hasan wants to merge 1 commit into
rust-lang:mainfrom
Albab-Hasan:cli-forbid-lowering-error
Open

Albab-Hasan wants to merge 1 commit into
rust-lang:mainfrom
Albab-Hasan:cli-forbid-lowering-error

Conversation

@Albab-Hasan

@Albab-Hasan Albab-Hasan commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

-F x -A x on the command line silently kept the forbid while the crate attribute form is E0453. now both error. groups hard error too instead of the forbidden_lint_groups fcw since the cli never let them lower a forbid.

the skip started as the #70918 hack for #70819 and #86970 moved it into add_command_line. related #133346 #81670

cc @jyn514

a later `-A`, `-W` or `--force-warn` for a lint forbidden by an earlier `-F` was silently ignored while the same order as crate attributes is E0453. command line levels now go through `insert_spec` like attributes so `-F dead_code -A dead_code` is E0453 too. lint groups also hard error instead of the `forbidden_lint_groups` future compat warning since the command line never let them lower a forbid.
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 9, 2026
@rustbot

rustbot commented Nov 9, 2026

Copy link
Copy Markdown
Collaborator

Error: Failed to set assignee to jyn514: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #triagebot on Zulip.

@jyn514

jyn514 commented Nov 9, 2026

Copy link
Copy Markdown
Member

r? compiler

@jyn514

jyn514 commented Nov 9, 2026

Copy link
Copy Markdown
Member

cc @folkertdev

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants