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

It's not possible to allow #![allow(text_direction_codepoint_in_comment)] #90614

Closed
KamilaBorowska opened this issue Nov 5, 2021 · 3 comments · Fixed by #90626
Closed

It's not possible to allow #![allow(text_direction_codepoint_in_comment)] #90614

KamilaBorowska opened this issue Nov 5, 2021 · 3 comments · Fixed by #90626
Assignees
Labels
C-bug Category: This is a bug.

Comments

@KamilaBorowska
Copy link
Contributor

KamilaBorowska commented Nov 5, 2021

I tried this code:

#![allow(text_direction_codepoint_in_comment)]

I expected to see this happen: text_direction_codepoint_in_comment to be allowed

Instead, this happened:

warning: unknown lint: `text_direction_codepoint_in_comment`
 --> src/main.rs:1:10
  |
1 | #![allow(text_direction_codepoint_in_comment)]
  |          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: did you mean: `text_direction_codepoint_in_literal`
  |
  = note: `#[warn(unknown_lints)]` on by default

(in case anyone is curious why I noticed this, don't worry, this isn't due to production code, see https://codegolf.stackexchange.com/a/230964/3103)

Meta

rustc --version --verbose:

rustc 1.56.1 (59eed8a2a 2021-11-01)
binary: rustc
commit-hash: 59eed8a2aac0230a8b53e89d4e99d55912ba6b35
commit-date: 2021-11-01
host: x86_64-unknown-linux-gnu
release: 1.56.1
LLVM version: 13.0.0
@KamilaBorowska KamilaBorowska added the C-bug Category: This is a bug. label Nov 5, 2021
@jyn514
Copy link
Member

jyn514 commented Nov 5, 2021

cc @estebank

@hkratz
Copy link
Contributor

hkratz commented Nov 5, 2021

@rustbot claim

@estebank
Copy link
Contributor

estebank commented Nov 6, 2021

That will show me not add proper test coverage 🤦‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants