proposal/process-review-no-code-managment-tasks common: added info by Danilqa · Pull Request #72 · eigen-space/codestyle

Danilqa · 2019-09-05T15:33:30Z

No description provided.

tohasan

Thank you! Great!

tohasan · 2019-09-05T19:02:24Z

doc/common/README.md

+Например:
+[Документ GoogleDocs](https://docs.google.com/spreadsheets/d/1yZT4qOec28r-uTDg9uz3TOvNi5I0ePbppZa7URWby1s/edit?usp=sharing)
+
+2. После того, как ревьюер завершил фиксирование замечаний, он уведомляет об этом автора задач.


I suggest to add also: Переводит задачу в статус "открыта" / "переоткрыта", чтобы другие ревьюеры делали обзоры на непросмотренные задачи.

tohasan · 2019-09-05T19:06:05Z

doc/common/README.md

+
+Такие задачи, как документирование в Notion, Confluence и пр. 
+
+1. Ревьюер в процессе обзора кода фиксирует свои замечания и пожелания в одном 


I suggest to use russian word in russian text. For instance, we can use either "рецензент" or "автор обзора" instead of "ревьюер".

Wow, a really great term, thanks!!

tohasan · 2019-09-05T19:06:18Z

doc/common/README.md

+4. После того, как автор задачи завершил улучшение проделанной работы и отметил все пункты как 
+   завершённые, он также сообщает об этом ревьюверу.
+
+5. Если ревьювер с чем-то не согласен, он отмечает незавершённые пункты и отдаёт их на доработку,


Suggested change

5. Если ревьювер с чем-то не согласен, он отмечает незавершённые пункты и отдаёт их на доработку,

5. Если ревьюер с чем-то не согласен, то процесс повторяется с шага 2, иначе задача переводится в состояние "готово".

tohasan · 2019-09-05T19:12:19Z

doc/common/README.md

+месте. \
+Например:
+[Документ GoogleDocs](https://docs.google.com/spreadsheets/d/1yZT4qOec28r-uTDg9uz3TOvNi5I0ePbppZa7URWby1s/edit?usp=sharing)
+


Suggested change

В данном случае мы также преследуем цель: сократить количество отвлекающих воздействий на автора изменений, чтобы меньше отвлекать друг друга от работы. В данном случае должно остаться всего одно воздействие в виде ссылки на страницу с комментариями.

tohasan · 2019-09-05T19:14:47Z

doc/common/README.md

+1. Ревьюер в процессе обзора кода фиксирует свои замечания и пожелания в одном 
+месте. \
+Например:
+[Документ GoogleDocs](https://docs.google.com/spreadsheets/d/1yZT4qOec28r-uTDg9uz3TOvNi5I0ePbppZa7URWby1s/edit?usp=sharing)


We also can use slack one of the following ways:

Comments to task in review channel. One comment - one message as we did before. Then author of changes mark each item with green checkmark.

Find special app for slack with checkboxes if we want checkboxes :)

…tion

tohasan

Thank you!! Splendid!

tohasan · 2019-09-08T16:05:38Z

doc/common/README.md

 затягивается и ревью.

-#### 6.4.4. Противоречия и споры в процессе обзора кода
+#### 6.4.4. Процесс обзора задачи, не связанных с системой управления кодом (Github, Gitlab)


Suggested change

#### 6.4.4. Процесс обзора задачи, не связанных с системой управления кодом (Github, Gitlab)

#### 6.4.4. Процесс обзора задач, не связанных с системой управления кодом (Github, Gitlab)

tohasan · 2019-09-08T16:20:10Z

doc/common/README.md

+
+1. Рецензент в процессе обзора кода фиксирует свои замечания и пожелания в одном 
+   месте. \
+   Например: тематический канал в Slack, с отключёнными уведомлениями. \


Suggested change

Например: тематический канал в Slack, с отключёнными уведомлениями. \

Например: канал `review` в Slack, где уже есть ссылка на задачу от автора изменений. \

Соответственно он получит уведомление об оставленном комментарии.

It is a little bit contradict with origin idea, but there is no problem if we activate special mode that described in comment above.

The bonuses:

there is no additional channel

we guard ourselves not only from review messages, but also from any other

we can use telegram for some exceptional cases (it should be very rare case)

tohasan · 2019-09-08T16:23:12Z

doc/common/README.md

+Такие задачи, как документирование в Notion, Confluence и пр. 
+
+Мотивация:\
+Сократить количество отвлекающих воздействий на автора изменений до 2 (в позитивном сценарии), 


We can solve it using mode Not Disturb Me (https://get.slack.help/hc/en-us/articles/214908388-Pause-notifications-with-Do-Not-Disturb) in Slack with specified schedule. We can create special rule for our team so that we have only a few time windows when we check out new messages and send responses.

tohasan · 2019-09-08T16:30:34Z

doc/common/README.md

+   А также переводит задачу в статус «открыта» / «переоткрыта», чтобы другие рецензенты делали 
+   обзоры только на непросмотренные задачи.
+
+3. Автор задачи знакомится со списком замечаний и пожеланий, называет список вопросов и обсуждает 


Suggested change

3. Автор задачи знакомится со списком замечаний и пожеланий, называет список вопросов и обсуждает

3. Автор задачи знакомится со списком замечаний и пожеланий, формирует список вопросов и обсуждает

tohasan · 2019-09-08T16:32:11Z

doc/common/README.md

+4. Автор задачи отмечает сделанные задачи смайлом: ✔ 
+
+5. После того, как автор задачи завершил улучшение проделанной работы и отметил все пункты как 
+   завершённые, он также сообщает об этом рецензенту.


I suggest to not use indent for all lines of list item because it is difficult to maintain it. See list item below.

… to tree quotes; updated rules

# Conflicts: # doc/common/README.md

proposal/process-review-no-code-managment-tasks common: added info

d551e34

Danilqa requested review from derlagg, sergeevNick and tohasan September 5, 2019 15:33

Danilqa self-assigned this Sep 5, 2019

proposal/process-review-no-code-managment-tasks common: updated info

64f5da8

tohasan approved these changes Sep 5, 2019

View reviewed changes

proposal/review-process-node-code-tasks common: fixed process descrip…

5834264

…tion

Danilqa requested a review from tohasan September 8, 2019 14:55

tohasan reviewed Sep 8, 2019

View reviewed changes

proposal/review-process-node-code-tasks common: removed double quotes…

1fe1a34

… to tree quotes; updated rules

Danilqa requested a review from tohasan September 9, 2019 08:26

Danilqa added 3 commits September 11, 2019 13:54

Merge branch 'dev' into proposal/review-process-no-code-tasks

34d70c4

# Conflicts: # doc/common/README.md

doc/process: supported english language

962f45b

doc/process: increased numbers of versions

e718c2a

sergeevNick force-pushed the dev branch from 90e36de to 1ca1a73 Compare September 12, 2019 11:57

doc/process: corrected en version of the rule about review non-repo task

183c2b9

tohasan merged commit d22e0df into dev Sep 12, 2019

tohasan deleted the proposal/review-process-no-code-tasks branch September 12, 2019 17:38


		Такие задачи, как документирование в Notion, Confluence и пр.

		1. Ревьюер в процессе обзора кода фиксирует свои замечания и пожелания в одном

	5. Если ревьювер с чем-то не согласен, он отмечает незавершённые пункты и отдаёт их на доработку,
	5. Если ревьюер с чем-то не согласен, то процесс повторяется с шага 2, иначе задача переводится в состояние "готово".

	#### 6.4.4. Процесс обзора задачи, не связанных с системой управления кодом (Github, Gitlab)
	#### 6.4.4. Процесс обзора задач, не связанных с системой управления кодом (Github, Gitlab)

	Например: тематический канал в Slack, с отключёнными уведомлениями. \
	Например: канал `review` в Slack, где уже есть ссылка на задачу от автора изменений. \
	Соответственно он получит уведомление об оставленном комментарии.

	3. Автор задачи знакомится со списком замечаний и пожеланий, называет список вопросов и обсуждает
	3. Автор задачи знакомится со списком замечаний и пожеланий, формирует список вопросов и обсуждает

Conversation

Danilqa commented Sep 5, 2019

Uh oh!

tohasan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Danilqa Sep 8, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tohasan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Danilqa Sep 8, 2019 •

edited

Loading