From 85931ab5946af3c35a3f982b0e5a3dae41c6ea23 Mon Sep 17 00:00:00 2001 From: Andrew Gallant Date: Fri, 22 Aug 2025 08:07:05 -0400 Subject: [PATCH] [ty] Add a TODO for linting on `todo!` --- crates/ty_python_semantic/src/types/infer.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/crates/ty_python_semantic/src/types/infer.rs b/crates/ty_python_semantic/src/types/infer.rs index 5d3b21cc7e..0cb21ae2f9 100644 --- a/crates/ty_python_semantic/src/types/infer.rs +++ b/crates/ty_python_semantic/src/types/infer.rs @@ -10040,6 +10040,11 @@ impl<'db> TypeInferenceBuilder<'db, '_> { // ================================================================================= // Branches where we probably should emit diagnostics in some context, but don't yet // ================================================================================= + // TODO: When this case is implemented and the `todo!` usage + // is removed, consider adding `todo = "warn"` to the Clippy + // lint configuration in `Cargo.toml`. At time of writing, + // 2025-08-22, this was the only usage of `todo!` in ruff/ty. + // ---AG ast::Expr::IpyEscapeCommand(_) => todo!("Implement Ipy escape command support"), ast::Expr::EllipsisLiteral(_) => {