Format on type fixes#1798
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1798 +/- ##
==========================================
- Coverage 74.1% 74.06% -0.04%
==========================================
Files 282 282
Lines 13089 13127 +38
Branches 2335 2348 +13
==========================================
+ Hits 9700 9723 +23
- Misses 3259 3269 +10
- Partials 130 135 +5
Continue to review full report at Codecov.
|
| import { ITextRangeCollection, IToken, TokenType } from '../language/types'; | ||
|
|
||
| const keywordsWithSpaceBeforeBrace = [ | ||
| 'and', 'as', 'assert', |
There was a problem hiding this comment.
Please add await as well (#1786 (comment)).
There was a problem hiding this comment.
A complete list of keywords can be found using https://docs.python.org/3/library/keyword.html#keyword.kwlist
There was a problem hiding this comment.
This is list of those that may have ( after them. Does not have to be complete.
| test('Dot operator', () => { | ||
| testFormatLine('x.y', 'x.y'); | ||
| testFormatLine('5 .y', '5.y'); | ||
| //testFormatLine('- 135 .bit_length()', '-135.bit_length()'); |
There was a problem hiding this comment.
Any particular reason why this test is commented out?
There was a problem hiding this comment.
I am yet to get to #1784, quite an edge case. The - part is covered by another test.
3 participants
Fixes #1793
Fixes #1790
Fixes #1788
Fixes #1782
Fixes #1781
Fixes #1780
Fixes #1763
This pull request: