Andrew Casey
8cbf42cff5
Fix lint errors
2017-08-22 11:33:34 -07:00
Andrew Casey
129ace5047
Update SymbolWalker tests
...
...to consume revised API.
2017-08-22 11:33:33 -07:00
Andrew Casey
f2eacc6395
Use Maps to store visited types and symbols
2017-08-22 11:33:33 -07:00
Andrew Casey
801c1f70a2
Reshape SymbolWalker API
...
1. Expose visited types and symbols
2. Automatically reset before each walk
2017-08-22 11:33:32 -07:00
Wesley Wigham
2c8a5c40b8
Make SymbolWalker internal
...
...until required by an external consumer.
2017-08-22 11:33:32 -07:00
Wesley Wigham
053b915061
Rebase SymbolWalker change onto master
...
From PR #9847 .
2017-08-22 11:33:32 -07:00
Ryan Cavanaugh
b217d96cc1
Merge pull request #17839 from RyanCavanaugh/ignoreScripts
...
Ignore scripts for types packages
2017-08-22 11:24:18 -07:00
Anders Hejlsberg
11c4c4cd61
Fix to use correct target type ID
2017-08-22 17:41:07 +01:00
Anders Hejlsberg
a3a2ff5f12
Optimize relations for type references with unconstrained type arguments
2017-08-22 10:13:08 +01:00
Anders Hejlsberg
2b10784415
Merge pull request #17922 from Microsoft/baseExpressionTypeParameters
...
Class type parameters not in scope in base class expression
2017-08-22 07:14:54 +01:00
Basarat Ali Syed
ac098535cb
export UsageEntry used by already exported functions 🌹 ( #17853 )
2017-08-21 16:55:40 -07:00
Andy
2b28916e5e
Simplify resolveBaseTypesOfClass ( #17918 )
2017-08-21 11:34:53 -07:00
Mohamed Hegazy
07e1d3b13d
Ensure string enums are generated in protocol.d.ts ( #17914 )
2017-08-21 09:44:03 -07:00
Anders Hejlsberg
35addce79e
Add comment
2017-08-19 09:46:02 +02:00
Andy
5e8e735db5
quickInfo: Don't check for type === undefined, check for any ( #17815 )
...
* quickInfo: Don't check for `type === undefined`, check for `any`
* Fixes:
* We still want to do some work even if type is `any`
* Second test for `if (type)` is necessary because it might not have been assigned.
2017-08-18 17:21:25 -07:00
Wesley Wigham
a136f554a7
Fix stack overflow when resolving default construct signatures ( #17878 )
...
* Fix stack overflow when resolving default construct signatures
* No need for || emptyArray
2017-08-18 17:20:57 -07:00
Benjamin Lichtman
45c62ac0a6
Merge pull request #17709 from uniqueiniquity/objectsInArray
...
Correct outlining spans for object and array literals in array
2017-08-18 16:25:08 -07:00
uniqueiniquity
e6c1afb4a0
Style changes and cleanup
2017-08-18 15:59:22 -07:00
Daniel Rosenwasser
7cc6bfc985
Only give suggestions when the name is an identifier.
2017-08-18 15:58:31 -07:00
Daniel Rosenwasser
50671c374c
Try to provide a spelling suggestion when object literals have excess properties.
2017-08-18 12:55:35 -07:00
Daniel Rosenwasser
70ad2bdb31
Ensure that property suggestions are correctly escaped.
2017-08-18 12:54:35 -07:00
Arthur Ozga
a3d113bed3
Merge pull request #16385 from aozgaa/isInMultiLineComment
...
multi-line comment formatting fix and handler
2017-08-18 12:43:37 -07:00
Wesley Wigham
ecd2ae8dac
Deduplicate inputfiles before running RWC tests ( #17877 )
...
* Deduplicate inputfiles before running RWC tests
We deduplicate in the compiler, but we don't in the harness - this causes tests where the same file is listed multiple times in the arguments to not have the expected errors written, because we write out the same file multiple times when we should not.
* Don't completely omit both copied of duplicates
* Remove trailing whitespace
* Maintain list order while filtering duplicates
* Merge adjacent loops
2017-08-18 11:44:36 -07:00
Arthur Ozga
6b68da1185
Revert "Fix getConstraintOfIndexedAccess"
2017-08-18 11:32:53 -07:00
Arthur Ozga
439cdca52f
Merge pull request #17870 from Microsoft/fix-getConstraintOfIndexedAccess
...
Fix getConstraintOfIndexedAccess
2017-08-18 11:32:48 -07:00
Andy
ade3b565ae
Revert public API changes to logger ( #17899 )
2017-08-18 11:20:07 -07:00
Anders Hejlsberg
fa6773e685
Type parameters from class should not be in scope in base class expression
2017-08-18 11:00:46 +02:00
Arthur Ozga
e4e969a210
respond to comments
2017-08-17 20:06:46 -07:00
Ron Buckton
fad97e369a
Remove debug assertions due to invalid syntax in generators transform
2017-08-17 17:32:06 -07:00
Andy
eef7d8bd3d
Treat explicit imports from node_modules as external library imports ( #16364 )
...
* Treat explicit imports from `node_modules` as external library imports
* Update baselines
2017-08-17 13:26:38 -07:00
Nathan Shively-Sanders
a187b17e97
Simplify mapped-type handling in computeBaseConstraint
2017-08-17 13:09:21 -07:00
Nathan Shively-Sanders
b8e0dedac0
Fix #17069 and #15371
...
1. `T[K]` now correctly produces `number` when
`K extends string, T extends Record<K, number>`.
2. `T[K]` no longer allows any type to be assigned to it when
`T extends object, K extends keyof T`.
Previously both of these cases failed in
getConstraintOfIndexedAccessType because both bases followed `K`'s base
constraint to `string` and then incorrectly produced `any` for types
(like `object`) with no string index signature. In (1), this produced an
error in checkBinaryLikeExpression`. In (2), this failed to produce an
error in `checkTypeRelatedTo`.
2017-08-17 12:40:10 -07:00
Andy
babb88a0aa
Remove duplicate function ( #17807 )
2017-08-17 06:52:15 -07:00
Daniel Rosenwasser
2729281a14
Merge pull request #17631 from Microsoft/synthesizedNamespaces
...
Always emit leading 'var' declarations for synthesized namespaces
2017-08-16 23:56:07 -07:00
Daniel Rosenwasser
a38da8af70
Merge pull request #17838 from Microsoft/rev2.6.0
...
Bump version to 2.6.0.
2017-08-16 23:50:50 -07:00
Arthur Ozga
bc0c9a440e
Merge branch 'master' into formattingAfterParseError
2017-08-16 20:27:06 -07:00
Arthur Ozga
760ef44c36
test if onlyMultiLine flag changes answer
2017-08-16 18:48:27 -07:00
Arthur Ozga
6029b5cce8
cleanup
2017-08-16 18:12:28 -07:00
Arthur Ozga
19e2fa6fc5
Merge branch 'master' into isInMultiLineComment
2017-08-16 17:57:04 -07:00
Arthur Ozga
b7bc7d889e
clarify JsxText handling
2017-08-16 17:56:20 -07:00
Arthur Ozga
23ca368020
Use simpler indentation for comments
...
* When in a multi-line comment, we would have liked to use the start of
the comment as a reference point for the indentation inside the comment,
but determining the number of columns shifted for the comment start
woudl require determining the length w/r/t graphemes, which we do not
currently implement. We would like to avoid taking on a runtime
dependency on a grapheme-parsing library.
Instead, we look at the indentation level on the previoud line or start
of the comment as a reference point, and correct shift for lines
starting with an asterisk.
2017-08-16 17:51:29 -07:00
Arthur Ozga
153b94aeb4
JsxText has no leading comments
2017-08-16 15:28:47 -07:00
Daniel Rosenwasser
b7020628c1
Addressed code review feedback.
2017-08-16 15:06:51 -07:00
Daniel Rosenwasser
3f50f20d0a
Updated version in 'src' as well.
2017-08-16 14:49:44 -07:00
Andy
d4fecd4e46
Have grammarErrorAtPos take the source file of its argument ( #17834 )
2017-08-16 14:48:58 -07:00
Andy
9bcbc97e14
Replace 'isSourceFileJavaScript(getSourceFileOfNode())' with 'NodeFlags.JavaScriptFile' ( #17835 )
2017-08-16 14:48:46 -07:00
Andy
54af8aa945
Array arguments to concat should be ReadonlyArrays ( #17806 )
2017-08-16 14:32:14 -07:00
Ryan Cavanaugh
af68a61ba7
Ignore scripts for types packages
2017-08-16 11:53:44 -07:00
Tycho Grouwstra
76fb6545a5
fix some copy-pasting error ( #17766 )
...
* fix some copy-pasting error
* update to reflect @ahejlsberg's feedback
2017-08-16 11:49:24 -07:00
Nathan Shively-Sanders
146f828919
Merge pull request #17710 from Microsoft/add-readonly-check-to-property-access-of-index-signature
...
Add readonly check to property access of index signature
2017-08-16 11:16:54 -07:00