Skip to content
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

ci: add react-18 to tests #194

Merged
merged 15 commits into from
Mar 6, 2023
Merged

ci: add react-18 to tests #194

merged 15 commits into from
Mar 6, 2023

Conversation

Yeti-or
Copy link
Contributor

@Yeti-or Yeti-or commented Oct 20, 2022

Реакт полностью переключен на 18 версию
Storybook теперь на react 18
Cypress + jest гоняются на двух версиях: 18 & 17

--
отключен стаб для Editable:selectText

🐤 Download canary assets:

colors--canary.194.4342208887.xml
color_metro_ios-swift--canary.194.4342208887.swift
color_metro_kotlin--canary.194.4342208887.kt
color_metro_react-native--canary.194.4342208887.ts
color_sberHealth_ios-swift--canary.194.4342208887.swift
color_sberHealth_kotlin--canary.194.4342208887.kt
color_sberHealth_react-native--canary.194.4342208887.ts
color_sbermarket_ios-swift--canary.194.4342208887.swift
color_sbermarket_kotlin--canary.194.4342208887.kt
color_sbermarket_react-native--canary.194.4342208887.ts
color_sberprime_ios-swift--canary.194.4342208887.swift
color_sberprime_kotlin--canary.194.4342208887.kt
color_sberprime_react-native--canary.194.4342208887.ts
color_selgros_ios-swift--canary.194.4342208887.swift
color_selgros_kotlin--canary.194.4342208887.kt
color_selgros_react-native--canary.194.4342208887.ts
color_smbusiness_ios-swift--canary.194.4342208887.swift
color_smbusiness_kotlin--canary.194.4342208887.kt
color_smbusiness_react-native--canary.194.4342208887.ts
PlasmaTokensColor--canary.194.4342208887.swift
shadow_sbermarket_react-native--canary.194.4342208887.ts
typo_mage_ios-swift--canary.194.4342208887.swift
typo_mage_kotlin--canary.194.4342208887.kt
typo_mage_react-native--canary.194.4342208887.ts
typo_plasma_ios-swift--canary.194.4342208887.swift
typo_plasma_kotlin--canary.194.4342208887.kt
typo_plasma_react-native--canary.194.4342208887.ts
typo_ruler_ios-swift--canary.194.4342208887.swift
typo_ruler_kotlin--canary.194.4342208887.kt
typo_ruler_react-native--canary.194.4342208887.ts
typo_sage_ios-swift--canary.194.4342208887.swift
typo_sage_kotlin--canary.194.4342208887.kt
typo_sage_react-native--canary.194.4342208887.ts
typo_sbermarket_ios-swift--canary.194.4342208887.swift
typo_sbermarket_kotlin--canary.194.4342208887.kt
typo_sbermarket_react-native--canary.194.4342208887.ts
typo_soulmate_ios-swift--canary.194.4342208887.swift
typo_soulmate_kotlin--canary.194.4342208887.kt
typo_soulmate_react-native--canary.194.4342208887.ts

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/plasma-b2c@1.152.2-canary.194.4342208887.0
npm install @salutejs/plasma-core@1.100.2-canary.194.4342208887.0
npm install @salutejs/plasma-hope@0.11.2-canary.194.4342208887.0
npm install @salutejs/plasma-icons@1.128.2-canary.194.4342208887.0
npm install @salutejs/plasma-temple@1.139.2-canary.194.4342208887.0
npm install @salutejs/plasma-tokens@1.38.1-canary.194.4342208887.0
npm install @salutejs/plasma-typo@0.31.1-canary.194.4342208887.0
npm install @salutejs/plasma-ui@1.172.2-canary.194.4342208887.0
npm install @salutejs/plasma-web@1.177.2-canary.194.4342208887.0
npm install @salutejs/plasma-cy-utils@0.45.1-canary.194.4342208887.0
npm install @salutejs/plasma-sb-utils@0.98.2-canary.194.4342208887.0
npm install @salutejs/plasma-tokens-android@2.45.1-canary.194.4342208887.0
npm install @salutejs/plasma-tokens-ios-swift@2.45.1-canary.194.4342208887.0
# or 
yarn add @salutejs/plasma-b2c@1.152.2-canary.194.4342208887.0
yarn add @salutejs/plasma-core@1.100.2-canary.194.4342208887.0
yarn add @salutejs/plasma-hope@0.11.2-canary.194.4342208887.0
yarn add @salutejs/plasma-icons@1.128.2-canary.194.4342208887.0
yarn add @salutejs/plasma-temple@1.139.2-canary.194.4342208887.0
yarn add @salutejs/plasma-tokens@1.38.1-canary.194.4342208887.0
yarn add @salutejs/plasma-typo@0.31.1-canary.194.4342208887.0
yarn add @salutejs/plasma-ui@1.172.2-canary.194.4342208887.0
yarn add @salutejs/plasma-web@1.177.2-canary.194.4342208887.0
yarn add @salutejs/plasma-cy-utils@0.45.1-canary.194.4342208887.0
yarn add @salutejs/plasma-sb-utils@0.98.2-canary.194.4342208887.0
yarn add @salutejs/plasma-tokens-android@2.45.1-canary.194.4342208887.0
yarn add @salutejs/plasma-tokens-ios-swift@2.45.1-canary.194.4342208887.0

@Yeti-or Yeti-or force-pushed the yeti-or.react-18 branch 3 times, most recently from ed5fc2e to 1d39a58 Compare October 27, 2022 07:51
@Yeti-or Yeti-or force-pushed the yeti-or.react-18 branch 2 times, most recently from e7a2d2a to 4436c51 Compare October 27, 2022 17:28
@salute-developers salute-developers deleted a comment from Salute-Eva Oct 27, 2022
@salute-developers salute-developers deleted a comment from Salute-Eva Oct 27, 2022
@Yeti-or Yeti-or force-pushed the yeti-or.react-18 branch 2 times, most recently from 839c364 to de033d2 Compare October 27, 2022 18:27
@Salute-Eva
Copy link
Contributor

@Salute-Eva
Copy link
Contributor

Component performance testing

Result: 🌚 Missing base report

Check out current report in job artifacts!

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-194/

@Salute-Eva
Copy link
Contributor

Component performance testing

Result: 🌚 Missing base report

Check out current report in job artifacts!

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-194/

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-194/

@Salute-Eva
Copy link
Contributor

Component performance testing

Result: 🌚 Missing base report

Check out current report in job artifacts!

@Salute-Eva
Copy link
Contributor

Demo canvas app deployed!

http://plasma.sberdevices.ru/pr/demo-canvas-app-pr-194/

@Salute-Eva
Copy link
Contributor

@neretin-trike
Copy link
Collaborator

добавишь небольшое описание? Про то что, тесты добавил, версию реакта переключил и т.д.

npm i
npm i react@17 react-dom@17 --prefix="./packages/plasma-ui"
npm i react@17 react-dom@17 --prefix="./packages/plasma-temple"
npm i react@17 react-dom@17 --prefix="./packages/plasma-hope"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

такс, а тут не надо ещё npm i react@17 react-dom@17 --prefix="./packages/plasma-core"? Там же тоже реакт имеется, и зависимые компоненты

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я решил что можно оставить только конечные пакеты чтобы оно быстрее ставилось, в плане реакт же резолвится там где испоняются тесты, думаю что кстати plasm-hope можно тоже удалить тут

npm i
npm i react@17 react-dom@17 --prefix="./packages/plasma-ui"
npm i react@17 react-dom@17 --prefix="./packages/plasma-temple"
npm i react@17 react-dom@17 --prefix="./packages/plasma-hope"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а тут нужен plasma-core?

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-194/

@Salute-Eva
Copy link
Contributor

Component performance testing

Result: 💀 Missing report, something went wrong

Check out current report in job artifacts!

@Salute-Eva
Copy link
Contributor

Demo canvas app deployed!

http://plasma.sberdevices.ru/pr/demo-canvas-app-pr-194/

@Salute-Eva
Copy link
Contributor

@Yeti-or Yeti-or merged commit 840877d into master Mar 6, 2023
@Yeti-or Yeti-or deleted the yeti-or.react-18 branch March 6, 2023 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants