Skip to content
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.

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

Attempts to clarify how to use global module definitions. #698

Merged
merged 2 commits into from Oct 7, 2018

Conversation

MicahZoltu
Copy link
Contributor

A common point of user confusion lies around module definitions. TypeScript compiles everything by default, so it can "discover" loose module definitions floating around the working directory. ts-node only compiles what is necessary, so it will not discover anything automatically unless it is part of the module search path (such as node_modules by default).

This documentation update tries to make it a bit more clear to readers exactly what they need to do in order to get global module definitions picked up by ts-node. Hopefully it will help reduce support requests related to global module definitions that "work fine with tsc" but "don't work with ts-node".

A common point of user confusion lies around module definitions.  TypeScript compiles everything by default, so it can "discover" loose module definitions floating around the working directory.  ts-node only compiles what is necessary, so it will not discover anything automatically unless it is part of the module search path (such as `node_modules` by default).

This documentation update tries to make it a bit more clear to readers exactly what they need to do in order to get global module definitions picked up by ts-node.  Hopefully it will help reduce support requests related to global module definitions that "work fine with `tsc`" but "don't work with `ts-node`".
@coveralls
Copy link

coveralls commented Oct 7, 2018

Coverage Status

Coverage remained the same at 85.101% when pulling a5b5bb0 on MicahZoltu:patch-1 into 3ff4a69 on TypeStrong:master.

@blakeembrey blakeembrey merged commit 77eff53 into TypeStrong:master Oct 7, 2018
@blakeembrey
Copy link
Member

Great, thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants