Python
/
ruff
mirror of https://github.com/astral-sh/ruff
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[`flake8-type-checking`] Disable TC006 & TC007 in stub files (#15179) 

Fixes: #15176

## Summary

Neither of these rules make any sense in stub files. Technically TC007
should already not have triggered, due to the typing only context of the
binding, but it's better to be explicit.

Keeping TC008 enabled on the other hand makes sense to me, although we
could probably be more aggressive with unquoting in a typing runtime
context.

## Test Plan

`cargo nextest run`
This commit is contained in:
David Salvisberg 2024-12-29 20:39:16 +01:00 committed by GitHub
parent bc3a735d93
commit f170932585
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
crates/ruff_linter/src/checkers/ast/analyze/bindings.rs
View File

@ -77,7 +77,7 @@ pub(crate) fn bindings(checker: &mut Checker) {
checker.diagnostics.push(diagnostic); checker.diagnostics.push(diagnostic);
} }
} }
if checker.enabled(Rule::UnquotedTypeAlias) { if !checker.source_type.is_stub() && checker.enabled(Rule::UnquotedTypeAlias) {
if let Some(diagnostics) = if let Some(diagnostics) =
flake8_type_checking::rules::unquoted_type_alias(checker, binding) flake8_type_checking::rules::unquoted_type_alias(checker, binding)
{ {

2
crates/ruff_linter/src/checkers/ast/mod.rs
View File

@ -1282,7 +1282,7 @@ impl<'a> Visitor<'a> for Checker<'a> {
if let Some(arg) = args.next() { if let Some(arg) = args.next() {
self.visit_type_definition(arg); self.visit_type_definition(arg);
if self.enabled(Rule::RuntimeCastValue) { if !self.source_type.is_stub() && self.enabled(Rule::RuntimeCastValue) {
flake8_type_checking::rules::runtime_cast_value(self, arg); flake8_type_checking::rules::runtime_cast_value(self, arg);
} }
} }