refactor: use in operator for membership tests - #30
refactor: use in operator for membership tests#30deepsource-autofix[bot] wants to merge 3 commits into
Conversation
This PR replaces redundant equality checks with concise membership tests using the `in` operator, improving code readability and reducing duplication.
- Consider using `in`: DeepSource identified patterns where two or more equality comparisons were chained with `or`. We updated the vote check to `self.total_votes in (self.yes_votes, self.no_votes)` and the message check to `message in ("Already migrated", "Already migrated and synced")`, streamlining these conditions into single membership tests.
> This Autofix was generated by AI. Please review the change before merging.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
 |
|
Overall Grade Focus Area: Hygiene |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Python |  |
Apr 21, 2026 10:13p.m. | Review ↗ |
| Rust | |
Apr 21, 2026 10:13p.m. | Review ↗ |
| Secrets | |
Apr 21, 2026 10:13p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
floss | b4587bb | May 25 2026, 06:00 PM |
1 participant
This PR replaces redundant equality checks with concise membership tests using the
inoperator, improving code readability and reducing duplication.in: DeepSource identified patterns where two or more equality comparisons were chained withor. We updated the vote check toself.total_votes in (self.yes_votes, self.no_votes)and the message check tomessage in ("Already migrated", "Already migrated and synced"), streamlining these conditions into single membership tests.