-
Notifications
You must be signed in to change notification settings - Fork 8
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
Совместимость с Angular 8: Новый EvoUiClass + миграция кода до ng8 #150
Conversation
@@ -0,0 +1,381 @@ | |||
import { Component } from '@angular/core'; | |||
import { ComponentFixture, TestBed, async } from '@angular/core/testing'; |
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.
мы пользуемся https://netbasal.gitbook.io/spectator
Сребейзил ветку от v3 |
@rsboarder @NorthFailer @andreymatveev какие то движения здесь будут? |
@thekip будут. На данный момент мы заняты проверкой в https://github.com/evotor/Evo-UI-Kit/tree/versions/v3 после завершения по v3 будем мержить этот PR |
@thekip ждём тестирования задач в которых используется данная версия кита. |
Я обновил все необходимые пакеты до 8, и проставил
{static: boolean}
где требовалосьЯ проверил визуально в сторибук. Компоненты работают как надо, имеющиеся тесты тоже проходят.
Я обновил EvoUiClass скопировав код оригинального ngClass из angular7, и адаптировав его для работы с префиксом. Я так же скопировал и полностью обновил весь тест сьют оригинального ngClass.
Необходимо так же обновить codelyzer до 5 версии.
Но там есть Breaking Changes https://github.com/mgechev/codelyzer/releases/tag/5.0.0
они переимеовали некоторые правила и это может сломать обратную совместимость там где используется этот же конфиг, но еще не обновлен codelyzer
Так же необходимо обновить
ngx-page-scroll
и у них так же есть Breaking Changes, и это уже за рамками этого PR. Nolanus/ngx-page-scroll#88 (comment)