Merge master into release-2.8 #22442

mhegazy · 2018-03-09T17:07:34Z

Ports:

Previously this only worked cross-file because it was a merge. Now it works anywhere, and locally it is actually binding a new property on the object literal symbol.

… invoked

…l being passed to the host)

…ries through watchFile

…s customisable

1. Add documentation 2. Better organisation of concerns in utility functions 3. Better handling of module.exports and exports in the binder's new code.

* Resolve config wildcards result order in a platform independent manner * Accept affected user test baseline * Per reccomendation, just change matchFiles * Add test asserting the same order on differing case sensitive platforms

* Add 'isNamedDeclaration' helper to reduce casts * Add assertion * Remove assertion

…e(node)) (#22387)

…cks (#22398)

Note that the error location is misleading because it's reported inside the merge step for the js initializer.

when the JS symbol is a JS initializer

…s-as-declarations More special declaration types in JS

…f replaceNode (#22404)

* Call process.stdout._handle.setBlocking(true) This prevents output from being truncated when the compiler can output errors to stdout faster than it can receive them. This may slow down performance for compilations for many errors, but those were already quite slow. * Disable tslint no-unnnecessary-type-assertion-2 It is wrong. It *is* necessary.

Different watchFile and watchDirectory options through environment variable

#22406)

* Treat 'yield;' as 'yield undefined;' * Use undefinedWideningType

…#22407) * annotateWithTypeFromJSDoc: Do less special-casing for arrow functions * Code review

* Fix #22133: Expose getOutliningSpans on the server protocol * Remove debugger statement from test

* Port generated lib files * Update line breaks

sandersn and others added 30 commits November 29, 2017 11:36

JS Object literal assignments are declarations

8bd66a0

Previously this only worked cross-file because it was a merge. Now it works anywhere, and locally it is actually binding a new property on the object literal symbol.

Test:js object literal assignment as declaration

c6a7751

Fix the error callback of the present directory watcher

f6f4652

Simplify and correct the fsWatchFile callback handling with the stats…

ed5673c

… invoked

Rearrange the code to have functions after executing code

ca52f08

Use toLowercase as utility function

63a3a65

Rename and restructure non polling watch file

7f32f9a

Simplify check output errors

c8e2b9c

Create polling interval as non optional parameter

cabb211

Update watchFile to take priority (which is currently polling interva…

aa5e49a

…l being passed to the host)

Watch based on dynamic polling priority frequency queue

976f330

Add test to verify timeout queues

c3b9904

Fix the api test failures

09caaa3

Move the polling settings to sys instead of watch utilities

fa8d4cb

Add option to watch using fs.watch instead of watching through polling

bcfa02f

Allow recursive directory watching on non supported file system

787c995

Add support to test when watching directories through watchFile

0c04fb1

Add tests for #19989, #20023

c3db9fa

Improve test to verify the count of callbacks for the watched directo…

ab17600

…ries through watchFile

Test case for watching using fs.watch recursively

f4954d0

Add watchDirectory to be using dynamic polling

56d754c

Merge branch 'master' into watchOptions

a74e54e

Merge branch 'master' into watchOptions

4eba27b

Merge branch 'master' into watchOptions

93c13ce

Add watchFile option to use fsEvents on parent directory

95b2630

Merge branch 'master' into watchOptions

ca4af4f

Make the PollingInterval, PollingChunkSize, PollingUnchangedThreshold…

b7f6910

…s customisable

Fix the lint error

c5b21d4

Merge branch 'master' into js-object-literal-assignments-as-declarations

4f07f58

Support var x = x || {} declarations in JS

9e8d59c

sandersn and others added 27 commits March 7, 2018 14:51

Merge branch 'master' into js-object-literal-assignments-as-declarations

d86440f

Address PR comments

239f214

1. Add documentation 2. Better organisation of concerns in utility functions 3. Better handling of module.exports and exports in the binder's new code.

Accept user baseline changes

a826c78

Use single replacer for string escaping (#22335)

45eaf42

Add 'isNamedDeclaration' helper to reduce casts (#22089)

2676786

* Add 'isNamedDeclaration' helper to reduce casts * Add assertion * Remove assertion

fixJSDocTypes: Use ChangeTracker (#22391)

0605424

Use getTextOfNode over getTextOfNodeFromSourceText(getSourceFileOfNod…

55bffba

…e(node)) (#22387)

Filter symbol property names out of index signature assignability che…

9586288

…cks (#22398)

Disallow JS/non-JS merge without crashing

04ceb3d

Note that the error location is misleading because it's reported inside the merge step for the js initializer.

Improve error span:duplicate symbols cross-js/ts

35730f2

when the JS symbol is a JS initializer

Merge branch 'master' into js-object-literal-assignments-as-declarations

f8134d0

Merge pull request #21974 from Microsoft/js-object-literal-assignment…

99d866d

…s-as-declarations More special declaration types in JS

Import types in JS with var x = require('./mod') (#22161)

e4610e3

annotateWithTypeFromJSDoc: Use changes.insertTypeAnnotation instead o…

a49e83f

…f replaceNode (#22404)

Directly assign values for watchFile and watchDirectory in node System

8378f69

Merge pull request #21243 from Microsoft/watchOptions

17b10dc

Different watchFile and watchDirectory options through environment variable

Handle resolving and unknown symbols in getLiteralTypeFromPropertyName (

87d88e2

#22406)

Factor out a getYieldedTypeOfYieldExpression helper (#22416)

28e8c4f

Treat 'yield;' as 'yield undefined;' (#22297)

e48bcd6

* Treat 'yield;' as 'yield undefined;' * Use undefinedWideningType

FunctionLike = SignatureDeclaration (#22365)

2170f6e

annotateWithTypeFromJSDoc: Do less special-casing for arrow functions (…

0f47c8a

…#22407) * annotateWithTypeFromJSDoc: Do less special-casing for arrow functions * Code review

Port changes from #22172, provide a callback to fs.close (#22405)

1fc3aeb

Fix #22133: Expose getOutliningSpans on the server protocol (#22400)

c335c38

* Fix #22133: Expose getOutliningSpans on the server protocol * Remove debugger statement from test

Port generated lib files (#22437)

3480bf2

* Port generated lib files * Update line breaks

Merge branch 'master' into release-2.8

260194c

mhegazy merged commit ffdefea into release-2.8 Mar 9, 2018

mhegazy deleted the mergeMaster3-9 branch March 9, 2018 17:19

microsoft locked and limited conversation to collaborators Jul 25, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merge master into release-2.8 #22442

Merge master into release-2.8 #22442

mhegazy commented Mar 9, 2018 •

edited

Loading

Merge master into release-2.8 #22442

Merge master into release-2.8 #22442

Conversation

mhegazy commented Mar 9, 2018 • edited Loading

mhegazy commented Mar 9, 2018 •

edited

Loading