Предложение перенести процесс проверки темы WordPress на GitHub

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

В настоящее время процесс обзора темы проходит на Trac . Для многих авторов и обозревателей тем программа может показаться устаревшей. У него нет набора функций, к которому разработчики привыкли в других решениях. С перемещением основных частей WordPress, таких как разработка функциональных плагинов, на GitHub за последние несколько лет, возможно, пришло время переоценить другие области основного проекта.

Чолжом отметил в предложении, что проверки, проводимые через Trac, являются громоздкими. Как бывший руководитель группы проверки темы и рецензент, я знаю, что чувствуют многие в команде. Сейчас большая часть процесса проверки выполняется вручную. Это достигается за счет постоянного обмена мнениями между рецензентом и автором. Нет хорошего способа оставить заметку или комментарий к определенной строке кода, когда есть проблема. Этой продолжающейся дискуссии между рецензентом и автором иногда препятствует языковой барьер. Опыт сравнения изменений между обновлениями кода в лучшем случае неутешителен. Кроме того, единственная автоматическая проверка выполняется плагином Theme Check .

Обработка обзоров на GitHub открывает новый мир возможностей, которые могут сделать обзоры более эффективными и обеспечить лучший опыт на обоих концах процесса.

С 2015 по 2019 год я вел побочный бизнес с партнером, где мы проводили обзоры кода над плагинами и темами. Большая часть этого бизнеса работала с тематическими коммерческими магазинами. Около 90% обзоров обрабатывались на GitHub.

В то же время я все еще был волонтером в группе проверки тем WordPress. Сравнение с пользовательским опытом не проводилось. GitHub победил. Этот опыт отпугнул меня от желания выполнять обзоры для официального каталога. Там ничего не было упорядочено. Было сложнее указать на конкретные проблемы кода и проверить, была ли проблема устранена, когда автор отправил обновление.

Первоначальное предложение oljom описывает систему, в которой автор темы отправляет свою тему через страницу загрузки темы WordPress . Если он пройдет первоначальную проверку, система автоматически создаст новый репозиторий GitHub с темой. Затем авторы темы могут разветвить этот недавно созданный репозиторий для работы над изменениями кода на основе отзывов рецензентов. Обновления тем также будут работать через GitHub.

«Я считаю, что вилы – хорошая золотая середина», – сказал Жолжом. «У авторов есть форки для собственных учетных записей GitHub, и они могут обновлять их, и даже делать обновления таким образом. Это определенно поможет нам с людьми, которые пытаются обмануть систему, обновляя свои темы, когда они начинают работать с вещами, которые нарушают требования ».

Предложение – это просто идея для изучения на данном этапе. Большая часть процесса может измениться, если ему будет дан зеленый свет.

«В конце концов, я надеюсь, что эта инициатива будет реализована и поможет нам сократить очередь на рассмотрение», – сказал Жолжом. Он надеется, что команда сможет интегрировать больше автоматизированных проверок, таких как стандарты кодирования WordPress . Благодаря большей автоматизации это также может означать, что группа проверки темы может сосредоточиться на других проектах, над которыми она работает, например, на подготовке к будущему полномасштабного редактирования сайта и непрерывного обучения.

Помимо процесса проверки

Одна из основных проблем, с которой команда столкнулась на протяжении многих лет, – это обучение авторов написанию более чистого и безопасного кода. Некоторые проблемы, такие как дезинфекция данных на входе и их экранирование на выходе, по-прежнему распространены во время проверок. История общения команды через Trac, похоже, в целом не помогла с обучением.

«Главное – это то, что многие люди могут быть вовлечены в процесс проверки, и тогда у вас есть опытные рецензенты, которые могут показать, почему какой-то код не подходит или требует улучшения», – сказал Жолжом. «Это дает возможность многим людям увидеть, что не так с кодом и как они могут его улучшить. Кроме того, разветвленная тема остается в их репозитории, поэтому они могут видеть, какие изменения они внесли и почему ».

У oljom есть несколько вещей в резерве для долгосрочного списка желаний. Одна из возможностей на будущее может заключаться в настройке тем в Packagist для установки через Composer. Он признает, что в лучшем случае это риск.

«Я вижу в этом возможность привести один аспект WordPress в один ряд с другими современными PHP-фреймворками, такими как Laravel», – сказал он. «Кроме того, мы используем имеющиеся в нашем распоряжении автоматические инструменты. Используя PHP CodeSniffer, PHPStan, ES Lint и т.д.».