Replies: 3 comments
-
Что решим в этой дискуссии? @furtivite @igsekor |
Beta Was this translation helpful? Give feedback.
0 replies
-
Это нормально. Он полез, потому что изменился файл словаря. Если словарь не меняется, то проверка просходит только по файлам пул реквеста. Думаю, что ок было бы зафиксить эту ошибку тут же или отдельным PR, который потом подмёржили в эту ветку |
Beta Was this translation helpful? Give feedback.
0 replies
-
@nlopin совершенно прав. При изменении словаря необходимо проверить все файлы, ибо важно понять, внесёт ли это какие-то проблемы в основную ветку. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
В пиаре #1274 спеллер полез в файл, который не правился в рамках пиара.
Было бы круто, если бы спеллер проверял только файлы из пиара во время мёржа.
Глобальную проверку можно запускать по таймеру в мейне экшном, который бы ставил задачу на код-овнера папки, если находит ошибку.
Beta Was this translation helpful? Give feedback.
All reactions