Исправление builder-а для удобства разработки

[dimasites]

Что оно делает?

Исправлено недоразумение, по которому файлы и конфиги сборщика лежали в корне папки ./_build/ при том, что в ./core/ и ./assets/ , всё разложено по папочкам, то есть есть подпапки с названием компонента. То есть изменение только добавляет папку ./_build/minishp2/ и перекладывает файлы туда.

Зачем это нужно?

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

Принятие данных изменений избавит разработчиков от ненужной рутины, но не повлияет на workflow тех разработчиков, кому это изменение не нужно.

Подробнее проблему я описывал в telegram-чате сообщества MODX RSC

no reason

[dimasites]

Если вы хотите использовать новый для проекта инструмент управления зависимостями, то предложите пожалуйста вариант отказа от уже существующего в проекте аналога - npm, естественно в коде. Предлагать одновременное использование двух инструментов одинаковой направленности считаю нелогичным

В данном случае я не предлагаю ничего нового. В проекте уже используется git и я воспользовался стандартной функцией submodule.

К тому же env gui не опубликован в composer, если он имелся в виду под «аналогом npm», т.к. для его подключения хватает возможностей git. К тому же, далеко не во всех MODX-пакетах, где применяется данная среда, используются менеджеры пакетов, но во всех используется git, поэтому использование более универсального инструмента (git) считаю предпочтительным, потому предлагаю именно то решение, которое по итогу наблюдаете в PR.

Напомню, что этим PR я лишь хочу упростить жизнь участников сообщества за счет более логичной организации структуры папок и небольшого скрипта, имеющего 2 функции:

1. Вывод списка пакетов (вот такой), которые хочешь собрать, со ссылками на buil-скрипты (если работаешь только с minishop2, будет одна ссылка)
2. Возможность сразу скачать пакет (она и до меня была в установщике, но работала не на всех хостингах т.к. ./core/ как правило закрыты от доступа по http. Старый вариант я оставил для обеспечения BC, чтобы не нарушать DX тех, кто ею пользуется)

Как видно из коммитов, я сначала добавил копию скрипта build-env, но после удалил её, и сделал всё красиво и правильно.

Благодарю за ревью! Готов также ответить на другие вопросы по PR.

[GulomovCreative]

@dimasites

Я свой запрос на изменения отменил, т.к. мой аргумент нерабочий, npm - пакетный менеджер для js, а ты предлагаешь php - модуль. Прошу прощения.