Update dictionary.txt

[iprst]

No description provided.

[iprst]

Прикол — сначала тут было пусто после добавление ревьюеров, однако когда я сам добавил комментарий внутри страницы с кодом, он прилетел сюда с пометкой Pending — ожидает проверки. Удобно. Можно прямо в коде пояснить, что сделано, и ревизор это сразу увидит здесь, даже не заходя в файл.

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

Нужно только добавить ревизора наверху меню справа этой страницы

И прокомментировать нужные строки кода, нажав на плюсик в редакторе

[audetv]

В прошлом году один мой коллега поправил меня и сказал, что слова предикция нет.
Что ж теперь есть)

[iprst]

Если слова нет, то что тогда поисковики находят с миллионами результатов? Надо немедленно прекратить это безобразие.

[audetv]

Прикол — сначала тут было пусто после добавление ревьюеров, однако когда я сам добавил комментарий внутри страницы с кодом, он прилетел сюда с пометкой Pending — ожидает проверки. Удобно. Можно прямо в коде пояснить, что сделано, и ревизор это сразу увидит здесь, даже не заходя в файл.

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

Нужно только добавить ревизора наверху меню справа этой страницы

И прокомментировать нужные строки кода, нажав на плюсик в редакторе

И ещё в настройках репозитория можно выставить условие, что только пул реквест принимается в главную ветку и только после code review, можно попробовать включить, я для других репозиториев настраивал branch protection, а в этом ничего не трогал оставил по умолчанию. Но тогда придётся делать через пул реквест, несколько лишних движений.: terratensor/svodd#112 (reply in thread)
Как вариант, можно включить в настройках.

[iprst]

И ещё в настройках репозитория можно выставить условие, что только пул реквест принимается в главную ветку и только после code review, можно попробовать включить, я для других репозиториев настраивал branch protection, а в этом ничего не трогал оставил по умолчанию.

Видимо это и есть сценарий, при котором изменения в файле не создают уведомлений в колокольчике. Вероятно это сделано для того, чтобы в некоторых процессах с частым обновлением файлов в доверительном окружении (в предельном случае в одиночку) можно было работать как без пулл-реквестов, так и без отправки уведомлений. Сохраняет время и не загрязняет колокольчик шумом.

[iprst]

@grigorovsckaya-sandra кстати вам должно было прилететь уведомление, что требуется проверить внесённые изменения. Выше описано что к чему.

[grigorovsckaya-sandra]

И прокомментировать нужные строки кода, нажав на плюсик в редакторе
не ясно, какой комментарий от меня требуется )

[iprst]

не ясно, какой комментарий от меня требуется )

Это пример сценария работы с файлами, такой, в котором можно сделать так, чтобы другие заведомо увидели, и если нужно отреагировали. Например, если при внесении каких-то данных сомневаетесь, можно обновить эти данные не через Commit Changes, а через Pull Request (показывал гифку), и в случае пул-реквеста можно назначить проверку от других участников проекта. Под проверкой подразумевается просто привлечение внимание к вопросу.

Когда вас кто-то вызвал на проверку, просто оцените, как понимаете, что спрашивают (в основном это «я правильно сделал?») и нужно просто «дать ответ» — для этого не обязательно писать комментарий, достаточно просто поставить подтверждение:

Вот вы здесь и есть pending reviewer — ожидаемый ревизор — вам нужно нажать кнопку подтверждения, то есть вы согласны с внесёнными изменениями, у вас нет дополнительной информации или возражений, и так далее.

На странице с изменённым файлом должна быть кнопка REVIEW CHANGES и после её нажатия внизу страницы будет кнопка согласия.