Andrew Casey
724f71cae1
Describe the innermost scope as "enclosing scope" for constants
...
Using the same description as for functions was confusing because the
extracted constant was inserted below the top-level of that scope.
2017-10-02 15:57:36 -07:00
Andrew Casey
ba5f09b66d
Localize 'in' in extraction description
2017-10-02 15:30:52 -07:00
Anders Hejlsberg and GitHub
884c72efb4
Merge pull request #18654 from Microsoft/strictFunctionTypes
...
Strict function types
2017-10-02 23:09:17 +01:00
Anders Hejlsberg
bff843a9c9
Improve error elaboration for invariant generic types
2017-10-02 13:39:55 -07:00
Andy and GitHub
eefe5c9706
Fix acquiring format options for getEditsForRefactor ( #18848 )
...
* Fix acquiring format options for getEditsForRefactor
* Add test
* Fix test description
* Use `executeCommandSeq`
2017-10-02 13:26:35 -07:00
Daniel Rosenwasser
b80b2eeaa3
Move first/last shortcuts closer to the end of the helper flags.
2017-10-02 13:23:57 -07:00
Sheetal Nandi
898559b4e5
Merge branch 'master' into watchImprovements
2017-10-02 12:34:34 -07:00
Sheetal Nandi
835153ba50
PR feedback
2017-10-02 12:22:05 -07:00
Ron Buckton
fb3168d46f
Merge branch 'master' into dynamicNames
2017-10-02 11:40:28 -07:00
Andy and GitHub
637ed57451
Fix crash when extending non-EntityNameExpression ( #18853 )
2017-10-02 10:39:03 -07:00
Andy and GitHub
e6980722a6
Fix bug: Report errors on extends expression in JS even if an @augments tag is present ( #18854 )
2017-10-02 10:33:53 -07:00
Ryan Cavanaugh and GitHub
141cd574c7
Merge pull request #18603 from RyanCavanaugh/pluginWork
...
Report external files in initial case for project load
2017-10-02 10:32:04 -07:00
Andy and GitHub
a075ba9828
getSuggestionForNonexistentSymbol: Add comment ( #18885 )
2017-10-02 10:23:37 -07:00
Andy and GitHub
d11172c86e
Use idText in a few more places ( #18842 )
2017-10-02 08:37:02 -07:00
Andy and GitHub
b883fcbfdd
Enable "space-within-parens" lint rule ( #18856 )
2017-10-02 08:36:26 -07:00
Andy and GitHub
d821bbf3f3
Simplify RulesMap construction ( #18858 )
2017-10-02 08:36:09 -07:00
Daniel Rosenwasser
a23d1bf3ea
Updated helper for marginally better minification.
2017-09-30 00:06:20 -07:00
Daniel Rosenwasser
86315ed411
Added test and adjusted reporting logic.
2017-09-29 22:01:00 -07:00
Andrew Casey
bcc93f2c08
Loosen restriction on usages in innermost scope
...
Now that we're smarter about where we declare the extracted local, we
can ignore usage problems in the innermost script - it'll always have
access to the same symbols as the extracted expression.
2017-09-29 18:00:27 -07:00
Andrew Casey
c08308a0f1
Reuse getSourceFileImportLocation
2017-09-29 17:37:39 -07:00
Daniel Rosenwasser
5da45fb6de
Addressed code review feedback.
2017-09-29 17:16:50 -07:00
Daniel Rosenwasser
08ef6e4bea
Accepted baselines.
2017-09-29 17:08:35 -07:00
Daniel Rosenwasser
966f370712
Use a better check.
2017-09-29 16:42:30 -07:00
Anders Hejlsberg
936f98d9ad
Addressing CR feedback
2017-09-29 16:01:14 -07:00
Andrew Casey
9097a07758
Improve insertion positions of extracted constants
...
...mostly by putting them closer to the extraction site.
They now also follow all leading comments in a file (unfortunately,
including the doc comment on the first declaration).
Bonus: Special case declaration lists - declare the new variable as part
of the list, rather than before it, in case it depends on an earlier
entry.
2017-09-29 15:52:56 -07:00
Andrew Casey and GitHub
7aee3a102c
Merge pull request #18831 from amcasey/AdditionSpecialCase
...
Eliminate special case for extracting from a binary operator chain
2017-09-29 13:10:28 -07:00
Andy and GitHub
f71930c473
Print list of available refactors when we can't find one ( #18843 )
2017-09-29 11:08:45 -07:00
Sheetal Nandi
9e08caebf5
Merge branch 'master' into watchImprovements
2017-09-29 10:43:05 -07:00
Andy and GitHub
683d6c7ddd
Add helper functions for using unescapeLeadingUnderscores ( #18793 )
...
* Add helper functions for using `unescapeLeadingUnderscores`
* More cleanup
2017-09-29 09:57:31 -07:00
Anders Hejlsberg
0756aa1753
Merge branch 'master' into strictFunctionTypes
2017-09-28 15:39:57 -07:00
Andrew Casey and GitHub
8e4c559f2c
Merge pull request #18829 from amcasey/JsExtractTests
...
Improve Extract* unit test coverage and correctness
2017-09-28 14:59:18 -07:00
Andrew Casey
41676248e5
Eliminate special case for extracting from a binary operator chain
...
1) It assumed left-associativity and was, therefore, wrong for (e.g.)
exponentiation.
2) Arguably, if a user selects `a + |b + c|`, they want to extract `b +
c`, not `a + b + c`. Not being able to do so is surprising (and we may
eventually want to allow it), but so is having the rest of the
least-common subtree extracted.
Fixes #18268
2017-09-28 14:54:52 -07:00
Anders Hejlsberg
5613be4907
Only methods and constructors are bivariant in --strictFunctionTypes mode
2017-09-28 14:33:35 -07:00
Andrew Casey
a73a553f58
Assert that Extract* baselines are syntactically valid
2017-09-28 14:31:35 -07:00
Andrew Casey
0d5d5cdf28
For any Extract* baseline that is valid JS, produce a JS baseline
2017-09-28 14:13:48 -07:00
Andy and GitHub
7959bd0a3d
Check JSDoc @param tag names ( #18777 )
2017-09-28 13:44:38 -07:00
Andy and GitHub
4bba6ee02e
Support accessing enum types from JSDoc ( #18703 )
2017-09-28 13:43:39 -07:00
Andy and GitHub
0b7dd5a4a5
Rename test baseline to be compatibile with windows ( #18827 )
2017-09-28 13:32:38 -07:00
Daniel Rosenwasser and GitHub
0ea7c61eb5
Merge pull request #18810 from Microsoft/unusedErrorArgs
...
Remove unused arguments for diagnostics
2017-09-28 12:51:57 -07:00
Armando Aguirre and GitHub
a39110add9
Merge pull request #18806 from armanio123/FixScriptBlockFormatting
...
Fixed formatting on script blocks
2017-09-28 12:35:32 -07:00
Andy and GitHub
1a2de721b5
Fixes to @augments handling ( #18775 )
...
* Fixes to @augments handling
* Renames and diagnostic changes
* Add test for < > characters
* Use more specific return type
2017-09-28 12:34:54 -07:00
Nathan Shively-Sanders
d797b4ab76
Correctly transform jsdoc parameter types
...
And give a better name for rest params
2017-09-28 11:40:56 -07:00
Andrew Casey
49d24fd89e
Rename extractMethod tests to extractFunction for consistency
2017-09-28 10:36:16 -07:00
Armando Aguirre
5225b40aab
Addedn rangeContainsRange helper function
2017-09-28 10:23:30 -07:00
Andrew Casey and GitHub
5f30106269
Merge pull request #18783 from amcasey/ExtractConstant
...
Initial implementation of Extract Constant
2017-09-28 10:09:44 -07:00
Andy and GitHub
a92d315eb6
Remove unnecessary cast ( #18822 )
2017-09-28 09:57:32 -07:00
Nathan Shively-Sanders
724a813105
Merge branch 'master' into refactor-jsdoc-types-to-typescript
2017-09-28 09:17:28 -07:00
Daniel Rosenwasser
76d92a5dd6
Remove unused arguments in program.ts.
2017-09-27 22:04:52 -07:00
Daniel Rosenwasser
b029857528
Give a more helpful error message when users try decorating using expressions that take no arguments.
2017-09-27 20:46:42 -07:00
Daniel Rosenwasser
b6629f4fac
Remove unused arguments.
2017-09-27 20:39:14 -07:00