-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
package: typescript-estreeIssues related to @typescript-eslint/typescript-estreeIssues related to @typescript-eslint/typescript-estreetriageWaiting for team members to take a lookWaiting for team members to take a look
Milestone
Description
Overview
Coming over from microsoft/TypeScript#54693 (comment): in v6 we removed parserServices.hasFullTypeInformation
. Consumers can now check if parserServices.program
exists. But, there are a few existing external consumers that rely on it:
- Sourcegraph search for
parserServices.hasFullTypeInformation
: 23 results - Sourcegraph search for
parserServices?.hasFullTypeInformation
: 4 results
If we do nothing and keep it removed, the referenced plugins/etc. may not work out-of-the-box with v6. I'm happy to send PRs over to help them upgrade, but that might be kind of a pain. And we don't know what internal code will break.
Proposal: how about we add it back in as a /** @deprecated */
property, then remove it in v7? It's not much to add/change on our end. Thoughts @typescript-eslint/triage-team?
Metadata
Metadata
Assignees
Labels
package: typescript-estreeIssues related to @typescript-eslint/typescript-estreeIssues related to @typescript-eslint/typescript-estreetriageWaiting for team members to take a lookWaiting for team members to take a look