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

feat: add locales generator #544

Merged
merged 9 commits into from
Feb 2, 2024
Merged

feat: add locales generator #544

merged 9 commits into from
Feb 2, 2024

Conversation

FrankJaskon
Copy link
Contributor

No description provided.

@FrankJaskon FrankJaskon self-assigned this Jan 31, 2024
@FrankJaskon FrankJaskon linked an issue Jan 31, 2024 that may be closed by this pull request
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

Посмотри так же посильно ли будет сделать тесты для этого пакета. Так же обязательно убедись что у тебя актуальный мастер-база - там обновились флоу

generators/locales/package.json Outdated Show resolved Hide resolved
generators/locales/package.json Show resolved Hide resolved
@FrankJaskon
Copy link
Contributor Author

@Nelfimov

Посмотри так же посильно ли будет сделать тесты для этого пакета.

Чот пока даже никаких идей, что мы можем протестировать.

@TorinAsakura
Copy link
Member

@FrankJaskon - получить название объекта

  • передать его в качестве аргумента для нового объекта в файле

Это как пример

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

давай проверим работоспособность еще так:

  • в корне проекта делаем postinstall
  • в него закладываем yarn workspaces foreach changed ... и выполняем твой скрипт

@FrankJaskon
Copy link
Contributor Author

@Nelfimov

давай проверим работоспособность еще так:

  • в корне проекта делаем postinstall
  • в него закладываем yarn workspaces foreach changed ... и выполняем твой скрипт

image

@Nelfimov
Copy link
Member

Nelfimov commented Feb 1, 2024

Этот скрипт следит за изменениями в воркспейсах. Если изменения есть - запускает то что ему скажешь.

Наша цель - сделать постинсталл скрипт, который будет срабатывать при запуске yarn в корне проекта, который будет просматривать изменения. Где изменения были, будет искать переводы и экстрактить их.

Важно помнить, что помимо воркспейсов с фронтом у нас есть и бэк, где переводы не нужны.

Тут есть пример использования этого скрипта: https://github.com/atls/tools#проверка-проекта-на-ошибки

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

проверить работу из корня

@FrankJaskon
Copy link
Contributor Author

@Nelfimov

Этот скрипт следит за изменениями в воркспейсах. Если изменения есть - запускает то что ему скажешь.
Наша цель - сделать постинсталл скрипт, который будет срабатывать при запуске yarn в корне проекта, который будет просматривать изменения. Где изменения были, будет искать переводы и экстрактить их.
Важно помнить, что помимо воркспейсов с фронтом у нас есть и бэк, где переводы не нужны.
Тут есть пример использования этого скрипта: https://github.com/atls/tools#проверка-проекта-на-ошибки

Никит, подскажи, пожалуйста, я пока не могу понять как аргументы в скрипт будут пробрасываться в зависимости от того, где именно у нас произошли изменения?

Как он будет определять где генерировать общую локаль, которая будет собирать в себя только входящие в энтрипоинт переводы?

@Nelfimov
Copy link
Member

Nelfimov commented Feb 1, 2024

Попробуй поиграй с этим скриптом, либо его аналогом - yarn workspaces foreach.... Например, yarn workspaces foreach prepack.

Этот скрипт пройдется по всем пакетам, и где найдет этот скрипт - запустит его. Ты можешь так же поступить с твоим скриптом - положить его в энтрипоинт фронтовых клиентов, а затем в корень добавить постинсталл

@FrankJaskon
Copy link
Contributor Author

@Nelfimov

Попробовал оба скрипта в лабе:

"postinstall": "yarn workspaces changed foreach --parallel run generate-locales" - отрабатывает нормально.

Если изменения внутри энтрипоинта, то пересобирает переводы только для этого энтрипоинта. Если изменения, например, в ui, то пересобирает переводы для всех энтрипоинтов(т.к все энтрипоинты зависят от ui). Если изменения, например, в gateway, то скрипты генерации переводов не вызываются вовсе.

yarn workspaces foreach - пересобирает все переводы при каждом вызове.

@Nelfimov
Copy link
Member

Nelfimov commented Feb 1, 2024

Для начала нам достаточно. Дополняй ридми и на ревью

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

поправь и запрашивай дальше ревью

generators/locales/package.json Outdated Show resolved Hide resolved
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

В остальном всё довольно неплохо

@TorinAsakura TorinAsakura merged commit 9ef7f95 into master Feb 2, 2024
8 checks passed
@TorinAsakura TorinAsakura deleted the feat/locales-generator branch February 2, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Добавить генератор переводов
3 participants