-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issueenhancementNew feature or requestNew feature or requestlocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.package: parserIssues related to @typescript-eslint/parserIssues related to @typescript-eslint/parser
Milestone
Description
Reference: import-js/eslint-plugin-import#2980
errorOnTypeScriptSyntacticAndSemanticIssues
is an undocumented parser option.
Specifically it's an option that's documented and accepted in typescript-estree
, but is not documented in parser
.
It was never intended for use within eslint runs and it only works as a parserOption
purely because we're lazy and we don't validate the options a user passes in.
We never wanted to report TS semantic errors from within our parser, and we have been building out syntax errors by hand within the conversion logic. As such we should outright ban this from being passed as a parserOption.
ljharb, peanutenthusiast and JoshuaKGoldberg
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issueenhancementNew feature or requestNew feature or requestlocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.package: parserIssues related to @typescript-eslint/parserIssues related to @typescript-eslint/parser