Issue #130: Грейд версий @atls/react-kratos-browser-flows

[Voiislav]

No description provided.

[Nelfimov]

Обрати внимание - твой пр не прошел проверки по линту и тайпчеку.

Ошибки связаны в большинстве с разными версиями @types/react. Унифицируй версию по всему проекту либо в ручную либо сделай резолюшн в корне.

Предпочтительнее второй вариант потому что остальные пакеты давно не обновлялись и могут пойти конфликты. Поэтому лучше поставить резолюшн на 17 реакт типы, но в package.json твоего пакета поставить 18 реакт типы.

[Voiislav]

@Nelfimov

но в package.json твоего пакета поставить 18 реакт типы.
C этим понятно - в package.json у kratos-browser-flows оставляю "@types/react": "18.2.31".

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

[Nelfimov]

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

я обычно просто вручную прописываю резолюшн в корневом package.json. Поставь только версию последнюю в мажорной 17.

[Voiislav]

Уточни, пожалуйста, по поводу резолюшна. Нужно выполнить команду yarn set resolution @types/react: 17.0.0 ?

я обычно просто вручную прописываю резолюшн в корневом package.json. Поставь только версию последнюю в мажорной 17.

Все готово, но ПР не прошел проверку на install этапе. Прописал резолюшн только в корневом package.json и оставил 18 версию в package.json kratos-browser-flows. В резолюшнах в yarn.lock вижу 18 версию. Ошибки, судя по всему, тоже ссылаются на это, скрин ниже. Я что-то упустил?

Details

UPD: решено

[Nelfimov]

1. @emotion не выше 11.9.Х
2. Тайпскрипт не выше 4.6.4
3. eslint не выше 8.18
4. После установки резолюшн надо обязательно проустановить зависимости через yarn
5. Нейминг ветки - надо соблюдать нашу конвенцию с префиксами названий - feat, fix, chore, doc и т.д.
6. Обрати внимание, что твой ПР затрагивает обновление всех пакетов, а в задаче - нужно грейдить только один. Поэтому во всех пакетах которые не указаны в задаче откатывай изменения.

[Voiislav]

1. @emotion не выше 11.9.Х
2. Тайпскрипт не выше 4.6.4
3. eslint не выше 8.18
4. После установки резолюшн надо обязательно проустановить зависимости через yarn
5. Нейминг ветки - надо соблюдать нашу конвенцию с префиксами названий - feat, fix, chore, doc и т.д.
6. Обрати внимание, что твой ПР затрагивает обновление всех пакетов, а в задаче - нужно грейдить только один. Поэтому во всех пакетах которые не указаны в задаче откатывай изменения.

Готово. Не использовал yarn upgrade-interactive, а обновил версии вручную в корневом и kratos-browser-flows package.json'ах. Проустановил зависимости через yarn. Но после yarn check возникла ошибка в другом пакете, текст ошибки:

packages/payment-widget/src/interfaces/fields.interfaces.ts
➤ YN0000: │
➤ YN0000: │ Error: Module '"react"' has no exported member 'HTMLInputTypeAttribute'.

Корневой резолюшн:

 "resolutions": {
    "@emotion/react": ">=11.9.0 <11.10.0",
    "@types/react": "17.0.2",
    "eslint": "<=8.18.0",
    "typescript": "<=4.6.4"
  },

@Nelfimov, подскажи, пожалуйста, в чем может быть причина ошибки в этом пакете? Или достаточно, чтобы yarn check не выдавал ошибки только в пакете kratos-browser-flows?

UPD: ошибок при yarn check больше нет, изменил в резолюшн "@types/react": "^17.0.0"

[Voiislav]

Совершил ошибку: после изменения имени ветки создал новый ПР вместо изменения текущего.

[Nelfimov]

Совершил ошибку: после изменения имени ветки создал новый ПР вместо изменения текущего.

Отменяй/сноси старый и работаем с верным.