dsherret / dsherret/ts-morph

fixUnusedIdentifiers is using remove all unused declarations and is dangerous

Open
#1,335 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
6.2k
Forks
238
Avg merge
2m
Merged PRs (30d)
1

Description

**Is your feature request related to a problem? Please describe.**

fixUnusedIdentifiers function is using typescripts build in remover, and platforms stopped supporting it because it is dangerous. as seen here:
https://github.com/microsoft/TypeScript/issues/30260

This function should either have a warning attached to it, removed, or a custom solution made. I think there is no good solution though.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.