TypeScript 2.x Support

[JamesHenry]

• Update typescript peerDependency to ^2.0.3 (Breaking: Updated to TypeScript 2.x (fixes #105) #112)
• Investigate and fix broken tests caused by updated dependency:
  • jsx/tag-names-with-dots (Breaking: Update JSX tagName conversion for TS 2 (refs #105) #108)
  • jsx/tag-names-with-multi-dots (Breaking: Update JSX tagName conversion for TS 2 (refs #105) #108)
  • basics/declare-namespace-with-exported-function (Fix: Exclude TSNode.flags when deeplyCopy fallback is used (refs #105) #107)
  • decorators/method-decorators/method-decorator-factory-instance-member (Fix: Rename decorator example to avoid issue in TS 2.x (refs #105) #106)
  • decorators/method-decorators/method-decorator-instance-member (Fix: Rename decorator example to avoid issue in TS 2.x (refs #105) #106)

---

• Transform new constructs introduced in TypeScript 2 (see TS 2.0 construct support #90 for additional background)
  • shorthand ambient module declarations (Chore: Added test for shorthand-ambient-module-declaration (refs #105) #109)
  • non-null assertion operator (Chore: Added test for non-null assertion operator (refs #105) #110)
  • typed usages of this (Breaking: Updated to TypeScript 2.x (fixes #105) #112)

---

• Add parameter type annotations to the AST (requisite for typed-this) (Fix: Add parameter type annotations to the AST (refs #105) #111)

[JamesHenry]

Update:

The decorators/method-decorators/method-decorator-factory-instance-member and decorators/method-decorators/method-decorator-instance-member tests fail because I just so happen to have called my made up decorator readonly, which became a keyword in TypeScript 2.

[JamesHenry]

The basics/declare-namespace-with-exported-function test fails because a particular flags value on the deeplyCopy()ed TSNode has changed between TS 1.x and TS 2.x

Whilst we do use TSNode.flags for some node checks, there is not actually any reason to pass them through to the converted AST when using the deeplyCopy() fallback. It does not currently provide any benefit, and we will always be exposed to these kinds of changes.

Therefore the change I will make is to exclude flags from the copy process, rather than changing the value in the final AST.

[JamesHenry]

jsx/tag-names-with-dots and jsx/tag-names-with-multi-dots were pretty tricky to track down and fix.

The change is AST between TS 1.8.10 and TS 2.0.3 can be attributed to this PR microsoft/TypeScript#7645

[dannyfritz]

Dang that was quick. ✨

[corbinu]

Nice work! as soon is up will run it through my Smoke Testing (obviously most will fail still) but hopefully when Eslint 4 drops we will be 100%!

[corbinu]

@JamesHenry Ran the smoke tests. No new failures.

https://gitlab.com/corbinu/eslint-typescript-smoketest/pipelines/4930979

All the failures are still in escope (as expected) except for the nativescript groceries sample which is failing on no-unused-vars and no-undef which is not unexpected either.

https://gitlab.com/corbinu/eslint-typescript-smoketest/builds/6041509

[JamesHenry]

Cool, thanks @corbinu! Now that TS 2 support is in, I'll be looking at the escope stuff