Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump TypeScript to v3.9 #67666
Bump TypeScript to v3.9 #67666
Changes from 65 commits
a82dea3
1209ac8
ac2bc71
d3cd8a8
090b1b7
31ce720
652af9b
870eb81
e3e68f3
be07879
8cae218
675a225
e168c62
97323f9
82ab3ef
8394514
9f68529
72f2296
e9ace30
dfa2574
dd7f2c8
f294214
e84eb25
3fe70e4
a482090
09d5706
4dd6549
796d839
1889b6c
991ddb3
dc3ce73
96771a2
f63dba3
9398916
c2a91ff
8490984
f77c5c3
9a99b84
4cdebff
f17e3b9
96fac96
997e7fa
8b2ecc8
a7bcdcf
626b710
baa6209
20e8e47
315c7a0
ad99021
efe7b08
dea0122
2b4a810
bc4ef14
c1d22c1
fe04a00
8fd55b2
a75a205
3caa679
ffe3338
3c7e50b
5a790f7
7b39e4f
f8b8e99
535afb3
cf406fb
beee8d2
3d39cd5
7579678
fa094a6
4113e83
888b90b
7951f77
403205c
90cd6d5
4a4bd7f
6b920ab
ff64040
02516af
68c5c91
a72a9e3
987b4a0
368197f
aac2380
b0b4d83
e2d6e7d
f25e739
b49dad5
fdb016c
3256c88
4ff4b54
16a8300
e98c40a
04d9594
64f08da
4218caf
e45065c
87a2846
babcc16
37e3263
cf62228
a887ff0
4baf675
d26ff2c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This syntax (and private member properties) seems like it might break api-extractor, or at least it's an untested syntax: microsoft/rushstack#1866 (comment)
Maybe we should hold off on making these changes until after this PR is merged to be sure we don't break compatibility with api-extractor (unless this was the only way to resolve a type error?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did the changes to test eslint parser update, we can roll them back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jloleysens Do you know if it's safe to remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
embeddable is defined in
props