generated from JoshuaKGoldberg/create-typescript-app
-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Labels
ai assignedA maintainer will resolve this issue with an AI agent.A maintainer will resolve this issue with an AI agent.plugin: tsThe plugin for TypeScript, a typed superset of JavaScript.The plugin for TypeScript, a typed superset of JavaScript.status: accepting prsPlease, send a pull request to resolve this! 🙏Please, send a pull request to resolve this! 🙏type: featureNew enhancement or request 🚀New enhancement or request 🚀
Milestone
Description
Feature Request Checklist
- I have pulled the latest
mainbranch of the repository. - I have searched for related issues and found none that matched my issue.
Overview
Per flint.fyi/rules, Flint is intended to have a unreachableStatements rule in the TypeScript plugin (ts). It'll behave roughly equivalently to the existing implementations in other linters to start. This issue tracks adding that rule.
Adding this rule will entail creating the following new source files:
packages/ts/src/rules/unreachableStatements.ts: implementation of the rule itselfpackages/ts/src/rules/unreachableStatements.test.ts: tests for the rulepackages/site/src/content/docs/rules/ts/unreachableStatements.mdx: documentation of the rule
Additionally, the following files will need to be edited:
packages/comparisons/src/data.json: Comparisons data will need to mention the rule is now"status": "implemented"packages/ts/src/plugin.ts: Included rules should have the new one inserted in alphabetical order
Additional Info
Existing rules in other linters:
- Biome:
noUnreachable - Deno:
no-unreachable - ESLint:
no-unreachable - Oxlint:
eslint/no-unreachable
❤️🔥
Metadata
Metadata
Assignees
Labels
ai assignedA maintainer will resolve this issue with an AI agent.A maintainer will resolve this issue with an AI agent.plugin: tsThe plugin for TypeScript, a typed superset of JavaScript.The plugin for TypeScript, a typed superset of JavaScript.status: accepting prsPlease, send a pull request to resolve this! 🙏Please, send a pull request to resolve this! 🙏type: featureNew enhancement or request 🚀New enhancement or request 🚀