Skip to content

Enhancement: Remove deprecated getTypeArguments from type-utils #8930

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
4 tasks done
JoshuaKGoldberg opened this issue Apr 15, 2024 · 1 comment
Closed
4 tasks done

Enhancement: Remove deprecated getTypeArguments from type-utils #8930

JoshuaKGoldberg opened this issue Apr 15, 2024 · 1 comment
Labels
accepting prs Go ahead, send a pull request that resolves this issue breaking change This change will require a new major version to be released enhancement New feature or request good first issue Good for newcomers locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.
Milestone
8.0.0

Comments

@JoshuaKGoldberg
Copy link
Member

Before You File a Proposal Please Confirm You Have Done The Following...

Relevant Package

type-utils

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

Back in #7834, we noted that the getTypeArguments function in type-utils is no longer necessary. It wrapped around a checker.getTypeArguments function added in TypeScript 3.7... which is well below the minimum version of TypeScript we support! The only reason we didn't delete the getTypeArguments function was that it would be a breaking change for API consumers.

But, we're working on a v8 major version - now's the time to delete the unnecessary code! 🔪

https://github.com/typescript-eslint/typescript-eslint/blob/96abf181410b7685dfdb0eb8313c43d1675881e2/packages/type-utils/src/getTypeArguments.ts

Additional Info

Note that the PR to delete this should be targeted against the v8 branch.

💖
@JoshuaKGoldberg JoshuaKGoldberg added enhancement New feature or request good first issue Good for newcomers breaking change This change will require a new major version to be released accepting prs Go ahead, send a pull request that resolves this issue labels Apr 15, 2024
@JoshuaKGoldberg JoshuaKGoldberg added this to the 8.0.0 milestone Apr 15, 2024
This was referenced Apr 17, 2024
Closed
Merged
@JoshuaKGoldberg
Copy link
Member Author

#8938 was merged into v8. ✅

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 30, 2024
@bradzacher bradzacher added the locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing. label Apr 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
accepting prs Go ahead, send a pull request that resolves this issue breaking change This change will require a new major version to be released enhancement New feature or request good first issue Good for newcomers locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.
Projects
None yet
Milestone
8.0.0
Development

No branches or pull requests
2 participants
@JoshuaKGoldberg @bradzacher