Skip to content

tsc moduleResolution default value Classic? #10433

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

Closed
wclr opened this issue Aug 19, 2016 · 5 comments
Closed

tsc moduleResolution default value Classic? #10433

wclr opened this issue Aug 19, 2016 · 5 comments
Assignees
Labels
Docs The issue relates to how you learn TypeScript

Comments

@wclr
Copy link

wclr commented Aug 19, 2016

TypeScript Version: 2.0.0 / nightly (2.0.0-dev.201xxxxx)

In docs https://www.typescriptlang.org/docs/handbook/module-resolution.html it is said

There are two possible module resolution strategies: Node and Classic. You can use the --moduleResolution flag to specify the module resolution strategy. The default if not specified is Node.

But if I run tsc the output is:
Module resolution kind is not specified, using 'Classic'.

@mhegazy
Copy link
Contributor

mhegazy commented Aug 20, 2016

the documentation is not accurate. The way it is setup is default is node, unless --module == Sysmte | AMD.

@mhegazy mhegazy added the Docs The issue relates to how you learn TypeScript label Aug 20, 2016
@mhegazy mhegazy self-assigned this Aug 20, 2016
@wclr
Copy link
Author

wclr commented Aug 20, 2016

right I had an "module": "amd"

@yahiko00
Copy link

Good to know! Because I felt that module resolution was a littble bit random ^^

@eemeli
Copy link

eemeli commented Oct 25, 2016

I hit the same issue today, a little over two months since this issue was first posted. Perhaps the documentation could be updated on this little detail?

@mhegazy
Copy link
Contributor

mhegazy commented Oct 26, 2016

Should be fixed by microsoft/TypeScript-Handbook@2426be0

alexinea added a commit to dotnetcore-internal/TypeScript-Handbook that referenced this issue Dec 29, 2016
* Add release notes

* Remove headers

* Add links to documentation for @types, --types and --typeRoots

Referenced in microsoft/TypeScript#11437 (comment)

* Update Compiler Options.md

Fix minor typo.

* Mention Babel at top of Gulp tutorial.

* IntelliJ Mac for nightly builds

Maybe include IntelliJ IDEA IDE for mac, too?

* Updated whitespaces and empty lines

* Use 'awesome-typescript-loader', use SFCs.

* Mentioned support for --outFile together with AMD or System

* Fixed broken link in React Webpack tutorial

 Updated the path reference to the page about tsconfig.json as it was broken.

* Fixes microsoft#420

clarifies Do's and Don'ts Use Optional Parameters section

See https://gist.github.com/agrberg/7bf64c61d3c6de0c638e7c0bc4742000 for
more details

* Change </br> to <br/>

* Fix microsoft/TypeScript#10433

* Fix microsoft/vscode#14322

* Fix paths example: target should be "jquery", not "jquery.d.ts"

Also add missing closing braces and have all json consistently use 2 space indents.

* Port fix in microsoft#432

* Fix lint failure

* Fixed typo

* Invoke origin setter in Decorators.md

* Add object rest/spread

* Update compiler options for TS 2.1

* Review comments

* Fix microsoft#348

* Review comments

* Add TypeScript 2.1 release notes

* Reorder

* Fix typo

* Minor typo

* Document index and indexed access types

* Document mapped types

* Reword and reorder index types discussion

* Fix module import

* Grammar fix

* Review comments

* Simplify keyof

* Fix typos

* Reorder mapped types example

Based on the more involved discussion from the blog post.

* Minor edits from PR comments

* Discuss mapped type inference and call out terms better

* Move index and map types to end of Advanced Types

The feature depends on type aliases and string literal types, so
should come after those.

* Document [non-]nullable types

* Does not require `this.` before props in the stateless functional component example

* `Hello` must be exported to be used in `index.tsx`

* Removed typo square bracket

* Minor formatting tweak from PR comments

* Mention there are currently 2 webpack loaders for typescript and link to more information

Fixes microsoft#450

* Fix defaults for `--importHelpers`

* Add breaking change link to release-notes

* Fix typo in variable reference

* Fix two small errors in React + Webpack tutorial.

* fixing #12872, add a mistype to show the usage of optional properties

* Fix typos

* Change the phrasing on webpack typescript loaders to be more clear

* Update Interfaces.md

* Update Interfaces.md

* limit files to just the src directory

* Update 'include' in React guide.
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Docs The issue relates to how you learn TypeScript
Projects
None yet
Development

No branches or pull requests

4 participants