-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Update to TypeScript 3.7 #3275
Update to TypeScript 3.7 #3275
Conversation
BAH! It appears prettier doesn't support the TypeScript 3.7 syntax features. It is fixed in master of prettier, but they haven't released it yet. |
It looks like prettier 1.19 is imminent which should include all the 3.7 syntax fixes, so I guess we should hold off on this until that is released. |
Could you raise an issue for the problematic asserts in std? |
[email protected] has been released now. |
Ok this now includes prettier 1.19 in If it passes it CI it should be ready to go @ry |
324e011
to
83607d6
Compare
@ry ok, I think I have fixed everything and this PR is ready now |
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.
LGTM - thank you @kitsonk !
Update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland#3273
Mistakenly re-enabled in denoland#3275.
Mistakenly re-enabled in #3275.
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland#3273
Mistakenly re-enabled in denoland#3275.
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
and update to prettier 1.19 Also, update `assert()` and remove not null assertions where possibly in `cli`. Closes denoland/deno#3273
Also, update
assert()
and remove not null assertions where possibly incli
. Lots of the not null assertions need to be removed instd
though.Closes #3273