-
Notifications
You must be signed in to change notification settings - Fork 799
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
Simplify TypeScript configuration (updated) #1028
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1028 +/- ##
=======================================
Coverage 86.54% 86.54%
=======================================
Files 31 31
Lines 825 825
Branches 193 193
=======================================
Hits 714 714
Misses 92 92
Partials 19 19 Continue to review full report at Codecov.
|
Looking great, but many people will refuse to use beta babel 7. |
Makes sense. Should we add warning to README and merge, or leave this PR until babel 7 becomes stable? |
I have no idea when it will be done, so lets try just some notes in README |
@theKashey are we good to proceed? |
All good, sorry just busy a bit in real world. |
I've found that some TS people do not like babel. They could be ok-ish to add babel as a webpack loader, but dont want to replace TS by babel as a whole. |
Quick work:
^
) babel dependencies because it's still in beta,master
,async
&await
syntax (EDIT: amended later),Though I should share this. Thanks @lostfictions for heavy lifting!