Skip to content

Conversation

@bkatsevych
Copy link
Collaborator

@bkatsevych bkatsevych commented Dec 8, 2023

Коротко:

  • Виправлено помилки
  • Додано 2 секції: Процес ревʼю перекладу та Відмінювання нестандартних слів
  • Внесено пропозиції стосовно перекладу певних слів глосарію

@alinkedd might be interested

@github-actions
Copy link

github-actions bot commented Dec 8, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@bkatsevych
Copy link
Collaborator Author

@alinkedd
Застосував запропоновані вами зміни.

@alinkedd
Copy link
Collaborator

alinkedd commented Dec 8, 2023

@bkatsevych я не можу резолвити, бо не мій pr і я не мейнтейнер, закрий, будь ласка, сам
і давай на ти, мені ще 30-ти нема :)

@bkatsevych
Copy link
Collaborator Author

bkatsevych commented Dec 8, 2023

@alinkedd
Я, як автор PR, не можу його аппрувити😭
Відправив Вам реквест на отримання адмін прав

@bkatsevych
Copy link
Collaborator Author

@alinkedd
Думаю, в такому разі зʼявляється потреба переписати процес ревʼю PRів, так як активних мейнтейнерів тепер 2

Моя пропозиція коротко - аппрувити по досягненню спільного консенсусу в результаті дискурсу

@alinkedd
Copy link
Collaborator

alinkedd commented Dec 8, 2023

@bkatsevych
можливо тоді хай автор після дискурсу і вносить правки (6 пункт)? а інакше після якогось часу (наприклад, тиждень) вже мейнтейнер?

так, може бути трохи довше, але з повноцінною асинхронною взаємодією, де автор доводить роботу до кінця і дізнається щось нове з самих правок, а мейнтейнер є редактором і кінцевим апрувером (один або обидва)

@bkatsevych
Copy link
Collaborator Author

bkatsevych commented Dec 8, 2023

@alinkedd
Прочитав уважніше.
Нехай так.

@bkatsevych
Copy link
Collaborator Author

@alinkedd
Аппрувніть, будь ласка

Copy link
Collaborator

@alinkedd alinkedd left a comment

Choose a reason for hiding this comment

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

lgtm

@alinkedd alinkedd merged commit fba76e6 into reactjs:main Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants