-
Notifications
You must be signed in to change notification settings - Fork 65
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 Angular to v10, TypeScript to 3.9 and use ESNext #2456
Bump Angular to v10, TypeScript to 3.9 and use ESNext #2456
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: maciaszczykm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #2456 +/- ##
==========================================
- Coverage 65.19% 57.06% -8.14%
==========================================
Files 283 283
Lines 11589 11165 -424
Branches 1460 1443 -17
==========================================
- Hits 7556 6371 -1185
- Misses 4022 4783 +761
Partials 11 11
Continue to review full report at Codecov.
|
3dbc8e9
to
6805984
Compare
Use Material Icons instead
Reenable semver imports
@kgroschoff It seems that I have find solution for the existing issues:
Please take a look :) |
/test test-e2e |
Build was failing because Subject was imported from rxjs/Subject instead from rxjs directly in a few places. Good to remember for the future. |
Nice work 👍 I only noticed one error in the browser console:
Do you also see it? Otherwise LGTM, couldn't find any other problems 👍 |
@kgroschoff It is fixed now. There was a reference to the old file. |
Great 😃 |
LGTM label has been added. Git tree hash: 073ec0acc580e3e680bccd15c605c77fbf963ffd
|
/hold |
/retest |
What this PR does / why we need it: https://volosoft.com/blog/what-is-new-in-angular-10
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #2448.Special notes for your reviewer:
Release note: