Skip to content

feat(ts): add unnecessary conditions rule - #3179

Draft
lishaduck wants to merge 2 commits into
mainfrom
feat/unnecessary-conditions
Draft

feat(ts): add unnecessary conditions rule#3179
lishaduck wants to merge 2 commits into
mainfrom
feat/unnecessary-conditions

Conversation

@lishaduck

Copy link
Copy Markdown
Member

PR Checklist

Overview

Adds the checker-backed ts/unnecessaryConditions rule for statically known truthiness, nullishness, comparisons, switch cases, optional chains, logical expressions, and built-in Array predicates.

Registers the rule in the logical preset, documents its semantic behavior and conservative exclusions, updates comparison data and end-to-end snapshots, and adds a patch changeset.

❤️‍🔥

@changeset-bot

changeset-bot Bot commented Aug 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6e5a338

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@flint.fyi/rule-data Patch
@flint.fyi/ts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify

netlify Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploy Preview for flint-fyi ready!

Name Link
🔨 Latest commit 6e5a338
🔍 Latest deploy log https://app.netlify.com/projects/flint-fyi/deploys/6a76aa971797a90008810e32
😎 Deploy Preview https://deploy-preview-3179--flint-fyi.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🚀 Feature: Implement unnecessaryConditions rule (TypeScript)

2 participants