Это неконструктивный диалог.
вы не произвели никакого ревью, вы позволили этому вообще дойти до этой отсутствующей стадии ревью
Anonymous
Anonymous
я говорю, что всё хуёво и должжно правится. И 90% заметно ПРОСТЕЙШЕЙ ВЫЧИТКОЙ 1им человеком.
Vadim
вы не произвели никакого ревью, вы позволили этому вообще дойти до этой отсутствующей стадии ревью
https://github.com/RussianFedora/rf-web/pull/85
Peter
вы не произвели никакого ревью, вы позволили этому вообще дойти до этой отсутствующей стадии ревью
ну и дальше то что? ты ж не хочешь его делать в рамках нашего workflow
Anonymous
вы говорите ревью прошло -> приняли и опуббликовали
Anonymous
ревью не было вообще, раз такое было опубликовано
Anonymous
какой здесь воркфлоу
Anonymous
было бы ревью - не пропустили бы это
Vadim
что самое смешное, фактически он прав - никто Петин текст там и не заревьювил
Peter
вы говорите ревью прошло -> приняли и опуббликовали
Так подпишись на активность репозитория и пиши +1 или минусуй.
Anonymous
тоесть код без тестов ушел в продакшн
Anonymous
норм?)
Aleksandra
@vrutkovs пока там участвует только один человек, делать ревью обязательным нельзя, потому что мы его будет по три месяца ждать
Anonymous
норм наверное.
Aleksandra
если будут регулярные читатели пулл-реквестов, то можно будет перейти на более строгий аппрув
Anonymous
@vrutkovs пока там участвует только один человек, делать ревью обязательным нельзя, потому что мы его будет по три месяца ждать
говорю, пингуйте меня, я сделаю, я исправлю все ошибки и тд
Vadim
@vrutkovs пока там участвует только один человек, делать ревью обязательным нельзя, потому что мы его будет по три месяца ждать
ради новости о предложении об изменении целей я бы скорее подождал
Aleksandra
тут нужно чтобы активность пошла и продержалась дольше двух дней после сегодняшней дискуссии
Peter
если будут регулярные читатели пулл-реквестов, то можно будет перейти на более строгий аппрув
Там только @ignatenkobrain читал иногда. Находил опечатки, кстати.
Anonymous
но в ваш отсутствующий воркфлоу я интегрироваться не собираюсь.
Peter
но в ваш отсутствующий воркфлоу я интегрироваться не собираюсь.
"Вот потому и Рашка"
Anonymous
Но больше меня удивляет, как взрослый человек не нашел 1000 ошибок при вычитке своего же текста
Anonymous
ДОДВАЙ ГАВАРИТЬ ВАТ ТАК
Anonymous
И В ГОЗЕТАХ ПЁЧАТАТЬ
Трайбор
Peter
Да, капец какой! :)
Anonymous
Меня порвало то, что это ушло в прод без вычитки
Anonymous
меня порвало от того, что петя переводил это и не увидел не состыковку ДАЖЕ С АНГЛИЙСКИМ текстом
Vadim
мне кажется его какое-то отдельное слово триггернуло
Anonymous
в "порвало" вы правы.
Aleksandra
давайте лучше про rm -rf
Vadim
надо Диму брать, вон какие нервы стальные, нам тут всем спуску не даст
Vadim
осталось научить его нажимать на кнопку Watch и заживем
Aleksandra
осталось научить его нажимать на кнопку Watch и заживем
это "осталось" - самая сложная часть нашей работы :)
Anonymous
Ответье лишь на 1 вопрос. Как это прошло ревью
Vadim
это "осталось" - самая сложная часть нашей работы :)
хотя ссылку на репу не мешало бы добавить вниз
Anonymous
Я нисколько не сомневаюсь, что никто ни разу не прочитал этот текст и опубликовал As is.
Vadim
Anonymous
ты видел ту кальку, с которой я сгорел изначально?
Anonymous
вот
Anonymous
Забавно однако перевели
Fedora creates an innovative platform that lights up hardware,
clouds, and containers for software developers and community
members to build tailored solutions for their users.
в
Проект Fedora создает инновационную платформу, которая включает в себя
и управляет оборудованием, облаками, и контейнерами. Создает для
разработчиков и участников сообщества, чтобы те могли создавать
специализированные решения для их пользователей.
Peter
это "осталось" - самая сложная часть нашей работы :)
Кстати, @ignatenkobrain уже предлагал спеллчекер прикрутить.
Vadim
ты видел ту кальку, с которой я сгорел изначально?
видел, вопрос в том можешь ли ты доказать свое утверждение
Peter
Может правда, пора aspell хотя бы добавить в юнит-тесты?
Aleksandra
@quko8 ты всё-таки не умеешь читать. я уже три раза объяснила
Aleksandra
spell-checker нельзя блокирующим делать
Anonymous
Даже худшие студенты филфака, издательского дела находят такие ошибки. У некоторых минимальный бал по русскому языку.
Peter
@vrutkovs ну да, там можно как aspell/ispell поставить и прогнать - там хотя бы главные проблемы будут видны. Можно, наверное, какой-нибудь web2/0 сервис подрубить.
Anonymous
А гуглдокс я предложил по той причине, что в нём удобнее рецензировать текст, давать дополнительные комментарии и одновременно группой лиц исправлять ошибки.
Aleksandra
можно добавить в тест, но он будет только warning выводить и это надо будет лезть далеко в лог чтобы прочитать эти warning-и
Anonymous
Aleksandra
spell-checker кому надо на стороне редактора лучше запускать, имхо
Aleksandra
может тебя замьютить на полчасика, пока не остынет
Anonymous
а из opendoc легко сделать маркдовн
Anonymous
Да делай что хочешь, я уже понял, что тебе похуй что принимать к публикации.
Anonymous
Совершенное непонимание специфики и различий кода и текста.
Peter
@bookwar кстати, а что с этим PR будем делать? Надо или принимать или отказывать уже.
https://github.com/RussianFedora/rf-web/pull/3
Anonymous
Петь, в дальнейшем ХОТЬ РАЗ прочитай текст после своей "кальки"
Anonymous
это не сложно.
Егор
Там строго 250 символов, ЕМНИП.
Егор
Я жеж его и писал.
Егор
Всё что есть сейчас, это максимум, что влезает.
Aleksandra
@peter_lemenkov закрыла с комментом
Aleksandra
а телеграм не умеет на пулл-реквесты подписываться?
Vadim
Может правда, пора aspell хотя бы добавить в юнит-тесты?
как-то всё плохо там - нужно дважды прогонять - для русского и английского - и false-positive сильно много
Vadim
Всё что есть сейчас, это максимум, что влезает.
а, сорри, я не про телеграмм канал, а про сам блог
Anonymous
Кто ответственен за вёрстку?