Microsoft/TypeScript

Move away from Classic module resolution as default for all module kinds

Open

#11.434 aberto em 6 de out. de 2016

Ver no GitHub
 (8 comments) (10 reactions) (0 assignees)TypeScript (6.726 forks)batch import
CommittedEffort: ModerateHelp WantedSuggestion

Métricas do repositório

Stars
 (48.455 stars)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (9 fundiu PRs em 30d)

Description

TypeScript's original module resolution strategy (the Classic resolution strategy) has some strange behavior that apparently doesn't actually reflect any other resolution strategy. It seems strange that those using non-CommonJS emit targets get this behavior by default.

Classic resolution also causes many disk probes during the walk up the project root - assuming that this strategy does eventually succeed in most cases, it seems like most uses could be subsumed by:

  • Using relative paths.
  • Modifying the baseUrl and paths properties in tsconfig.json.

After discussing with @vladima, we think it is worth considering switching the default resolution strategy for all module targets to Node.

Here's what needs to be done:

  • Investigate existing tsconfig.jsons that target esXXXX` and have no explicit/implied resolution strategy.
  • Determine the downstream effects to a change to defaults
  • Determine if this change outweighs the potential breakage
  • Implement if appropriate

Guia do colaborador