-
-
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.
Description
Before You File a Proposal Please Confirm You Have Done The Following...
- I have searched for related issues and found none that match my proposal.
- I have searched the current rule list and found no rules that match my proposal.
- I have read the FAQ and my problem is not listed.
Relevant Package
typescript-estree
My proposal is suitable for this project
- I believe my proposal would be useful to the broader TypeScript community (meaning it is not a niche proposal).
Description
Coming over from un-ts/eslint-plugin-import-x#130 (comment): some folks need to use withoutProjectParserOptions
on options that are type TSESLint.ParserOptions
rather than TSESTreeOptions
. But right now the function only allows the latter:
typescript-eslint/packages/typescript-estree/src/withoutProjectParserOptions.ts
Lines 11 to 13 in 3c3370f
export function withoutProjectParserOptions( | |
opts: TSESTreeOptions, | |
): TSESTreeOptions { |
Proposal: let's make the function generic so it can return the same type it's given?
Vaguely:
export function withoutProjectParserOptions<Options extends TSESTreeOptions>(
opts: Options,
): Omit<
Options,
'EXPERIMENTAL_useProjectService' | 'project' | 'projectService'
>
Additional Info
💖
SukkaW
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.