Document literal-membership fix safety conditions (#9677)

charliermarsh · web-flow · commit 05a2f5220697 · 2024-01-29T17:48:40.000Z
## Summary This seems safe to me. See #8482 (comment). ## Test Plan `cargo test`
diff --git a/crates/ruff_linter/src/rules/pylint/rules/literal_membership.rs b/crates/ruff_linter/src/rules/pylint/rules/literal_membership.rs
@@ -21,6 +21,12 @@ use crate::checkers::ast::Checker;
 /// ```python
 /// 1 in {1, 2, 3}
 /// ```
+///
+/// ## Fix safety
+/// This rule's fix is marked as unsafe, as the use of a `set` literal will
+/// error at runtime if the sequence contains unhashable elements (like lists
+/// or dictionaries).
+///
 /// ## References
 /// - [What’s New In Python 3.2](https://docs.python.org/3/whatsnew/3.2.html#optimizations)
 #[violation]
