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.

Sign up for GitHub

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

Jump to bottom

Add `assertions_on_constants` #81

Open

0xLucqs opened this issue Sep 18, 2024 · 1 comment

Labels

good first issue lint

Contributor

0xLucqs commented Sep 18, 2024 •

edited

Loading

What it does

Checks for assert!(true) and assert!(false) calls.

Why is this bad?

Should probably be replaced by a panic!() or removed

Example

assert!(false)
assert!(true)
const B: bool = false;
assert!(B)

0xLucqs added good first issue lint labels

Contributor Author

0xLucqs commented Sep 18, 2024

Sure

0xLucqs assigned KevinMB0220

0xLucqs unassigned KevinMB0220

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