Pull to refresh
15
0
Роман @RomeroMsk

User

Send message

И снова никаких изменений по VB-85124, которому уже больше года?

Не за что. Надеюсь, он поможет сделать продукт лучше.

Автоматическое добавление ссылки на PR GitHub и возможности оставлять произвольное описание к ревью (возможно, автоматически копируемое из описания PR) очень сильно упростят жизнь ревьюверам и мержерам. Думаю, после недавнего появления удобного сайдбара проблем с размещением этих новых элементов возникнуть не должно ;)

От переезда с GitHub останавливает отлаженная годами работа экшенов (CI/CD) и прочие фичи Enterprise-аккаунта, которыми активно пользуется несколько команд в компании. Убедить руководство потратить ресурсы на переезд - будет очень не просто. И весомых преимуществ никто не видит.

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

  • Неудобства, требующие бОльших затрат времени на проведение ревью:

    • [Issue] Автор ревью не назначается автоматически, когда оно создается из PR в GitHub. PR в GH нам по-прежнему нужны, т.к. на них построен CI/CD. И создавать ревью вручную (не автоматически из PR) каждый раз будет неудобно

    • [Issue] Автоматически не выбираются комиты, сделанные после твоего последнего акцепта/запроса изменений

    • [Issue, duplicated] Нет дескрипшена код ревью, куда хотелось бы вставлять ссылку на таск-трекер (при автоматическом создании ревью из PR описание тоже не синхронизируется, как было в Upsource). Приходится кидать ссылку в Timeline, где ее выискивать глазами не удобно, особенно когда чат уже сильно разросся

    • [Issue] Нет возможности перейти из ревью в PR в GitHub (например, чтобы проверить экшены)

    • [Issue, duplicated] Скролл сквозь все файлы может привести к случайному пролистыванию файла. Плюс эта фича не позволяет удобно прыгать между файлами, т.к. обратно возвращаешься в начало, а не запомненную позицию. Поштучный просмотр файлов, как в Upsource, в виде настраиваемой опции был бы идеальным решением

  • Проблемы интеграции с другими инструментами:

    • При мерже ревью оригинальный PR в GH просто закрывается (причем, от имени юзера-владельца репозитория в Space). В JIRA затем такие PR отображаются как DECLINED. В качестве решения мержим PR в самом GH и не пользуемся кнопкой мержа в Space

    • Мержи напрямую в Space - в целом, не очень удобный подход при работе с GitHub. В частности, в списке коммитов в GH при деплое не будет ссылок на PR для всех комитов, смерженных через Space

    • Чтобы заработала интеграция с PhpStorm, приходится клонировать проект заново из ssh://git@git.jetbrains.space/....git (существующий в PhpStorm проект, залинкованный на GitHub репозиторий - не распознается плагином)

  • Проблемы интерфейса (возможно, частично исправлены):

    • Если обсуждения скрыты, то после нажатия на иконку любого из них невозможно снова скрыть его

    • Усложняется восприятие диффа, если включить Split View и смотреть на код с длинными строками: перенос строк порождает новые (ненумерованные) строки в диффе

    • Стрелки для перехода к следующему найденному совпадению - перестают работать и снова начинают, если переключаться между Unified и Split View

    • Новые файлы попадают в область поиска при включенной опции Left, хотя ожидается, что Left/Right - это до/после

    • Если список изменений довольно большой, при сворачивании одной из папок в дереве файлов перестает корректно синхронизироваться позиция скролла в дереве файлов с панелью просмотра изменений

    • Не запоминаются свернутые папки в дереве файлов, если его скрыть/развернуть

  • Мелочи:

    • Нет возможности увидеть, когда ревьювер смотрел ревью в последний раз и сколько файлов ему еще осталось

    • Нет возможности в один/два клика скопировать путь к файлу в ревью

В статье упоминается JetBrains Space, но в голосовании его почему-то нет. Поделюсь опытом.

Мы прошли примерно тот же путь, когда поняли, что и без того не идеальный Upsource окончательно умирает. Только выбор пал на Space, плюс мы держим код в GitHub. Тоже в "боевом режиме" на нескольких проектах опробовали его, нашли положительные и отрицательные моменты и приняли решение об окончательном переезде. Тоже понадобилось время на привыкание и перестраивание некоторых привычек. Вот уже 4 месяца пользуемся. Несмотря на некоторые недостатки, общее впечатление от проведения код ревью в Space весьма положительное, для нашей команды он точно суммарно удобнее, чем Upsource.

переход на Space всей компании невозможен — он требует значительного увеличения бюджетов, и многие процессы уже настроены на использование GitLab

Если от всего "комбайна" Space больше особо ничего не нужно и лимиты функционала устраивают, код ревью можно пользоваться бесплатно. Нам вполне хватает, кроме код ревью ничего пока не заинтересовало, код всё так же в GitHub (где настроены экшены для CI/CD).

К сожалению, фича-реквесты закрываются весьма не быстро, некоторые висят без реакции по несколько месяцев. Но даже текущий функционал вполне радует.

Плохо, что за 4 месяца баг так и не брали в работу :( Учитывая другие встретившиеся баги последних версий, лично для меня нововведения 5-ки не перевешивают добавленных проблем.

Тем не менее, спасибо за отличную альтернативу Chrome! Зафиксирую для себя предыдущую мажорную версию тогда.

VB-85124 всё так и не исправлен? Придется до 4.х версии откатываться, видимо. Очень мешают эти проблемы с попапами :(

Это я понял, спасибо. Я отвечал автору комента выше на это:

Проверьте, может и у вас что-то лучше станет

Нет, лучше не стало. Ссылки из попапа по-прежнему открываются в том же окне, а не в основном окне браузера :(

Скажите, а проблему с popup'ами (VB-85124) так и не решили? Очень досадный баг 5 версии, из-за которого сильно хочется откатиться до 4 :( А тут уже и 5.1 с новыми фичами подъехала.

Я купил "балансировочную подушку" по совету своего массажиста. Вот такую: https://www.decathlon.ru/podushka-backmobility-id_8573556.html

Смысл в том, что она позволяет телу принимать более "правильное" положение, выравнивая (балансируя) его.

Попутно такая подушка решила и проблему жесткости для пятой точки. Причем, меняя степень надува, можно варьировать жесткость.

Еще очень «весело» разрабатывать всевозможные компоненты с учетом RTL. Например, datepicker, где направление дней недели, дат, месяцев, годов — тоже «обратное». Нажимаешь на кнопку пролистывания месяца вправо, а отображается предыдущий месяц:

Пока не работаю с удаленными вакансиями

Жаль. Дайте знать, когда начнете. Например, ответом на этот комментарий.
Спасибо за сервис и статью! Не совсем понятно, работаете ли вы с удаленными разработчиками.
Небольшой фидбэк: в «Terms of Service» фигурирует отличное от itrabota.com название сайта и не работают ссылки в шапке контента страницы.
Тоже проходили в своё время через осознание важности и полезности этого инструмента, который постепенно дал нам не только возможность удобного использования логов, но и мониторинг и аналитику. Кстати, его можно интегрировать с такими штуками, как Zabbix, если встроенного функционала перестает хватать — достаточно написать запросы напрямую к ElasticSearch или задействовать REST API.
Радуют темпы развития и качество продукта. Помню, когда пару лет назад обновились с 1 на 2 версию, пребывали в эйфории (особенно от обновленного веб-интерфейса).
Огромное спасибо за труды! С игрой знаком достаточно хорошо (правда, в классическую версию давно не играл — сейчас, в основном, в «Случайные мутации» рубимся). Сыграли партию, протестировали. Из багов бросились в глаза неформатированные сообщения об ошибках (названия свойств не подставляются). Долго не могли найти чат после старта игры — надо визуализировать как-то получше его. Остальные нюансы, принимаемые сперва за баги, оказывались следствием нашей невнимательности (пожалуй, стоит поработать над более подробными пояснениями при ошибках или запретах действий — например, сделать очевиднее причины невозможности «прицеливания» хищником на животных с определенными свойствами). Короче говоря, над оформлением/UI/UX можно поработать, но в целом все круто! Надеюсь, что не забросите это дело.
Локализация поддерживается для тех провайдеров, которые такую возможность предоставляют: http://geocoder-php.org/Geocoder/#locale-aware-providers
Не смотрели в сторону Geocoder? Мне думается, он мог бы помочь в реализации вашего пакета.
То есть Total Commander, Nero Burning Rom, WinRAR, NuSphere PhpEd и другие — недостаточно «крупные» проекты по вашему?
Я не имел в виду сравнение ради выяснения, кто точнее. Достаточно большое расхождение координат (более Х метров, скажем) из двух источников — повод для ручного разбора данного кейса и/или накопления статистики таковых расхождений.

Information

Rating
Does not participate
Location
Россия
Registered
Activity