Skip to content

Integration/open prs 20260906 - #650

Closed
AgelxNash wants to merge 19 commits into
modx-pro:betafrom
AgelxNash:integration/open-prs-20260906
Closed

Integration/open prs 20260906#650
AgelxNash wants to merge 19 commits into
modx-pro:betafrom
AgelxNash:integration/open-prs-20260906

Conversation

@AgelxNash

Copy link
Copy Markdown

Описание

Краткое описание изменений и их цели.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите):

Связанные Issues

Closes #(номер issue)

Как это было протестировано?

Опишите тесты, которые вы провели для проверки изменений.

Локальный CI-гейт (без полной установки MODX/MySQL), PHP lint + vueManager jobs из .github/workflows/ci.yml:

cd core/components/minishop3
composer install
composer ci:php

cd ../../../vueManager
npm ci
npm run lint:ci

PHPStan — отдельный job: composer stan:prepare && composer stan (pinned MODX/pdoTools в .phpstan-deps).

  • Ручное тестирование
  • Автоматические тесты (composer ci:php / composer test, npm run lint:ci, composer stan / GitHub Actions CI)
  • Тестирование на разных версиях PHP/MODX

Конфигурация тестирования:

  • MiniShop3:
  • MODX:
  • PHP:

Скриншоты (если применимо)

До После

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок (composer stan / CI job PHPStan)
  • ESLint проходит без ошибок (npm run lint:ci для Vue)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

Любая дополнительная информация для ревьюеров.

AgelxNash and others added 19 commits September 6, 2026 08:27
Task journal: fork created (AgelxNash/MiniShop3), all PR refs fetched, each PR applied as a single squash commit, conflicting PRs skipped and listed below. Our integration changes come first.

Merged: 596 598 600 614 619 620 623 624 627 629 631 637 639 642 643 647 648 649. Skipped (conflicts): 599 603 604 605 621 633 638 640 644 646. Drafts excluded. This PR itself (modx-pro#650) is excluded.
@AgelxNash
AgelxNash force-pushed the integration/open-prs-20260906 branch from 2617b2b to 4e4c37c Compare September 6, 2026 06:28
@AgelxNash

Copy link
Copy Markdown
Author

Что это за PR. Это не обычный фикс/фича, а тестовая консолидация: собраны все открытые PR репозитория на 06.09.2026 в одну ветку, чтобы проверить их совместную работу одной сборкой.

Как проверить:

git fetch https://github.com/AgelxNash/MiniShop3 integration/open-prs-20260906
git checkout FETCH_HEAD

По всем изменённым PHP-файлам пройден php -l; полноценные тесты не запускались — ветка именно для ручного тестирования.

Если такой мега-PR неудобен для ревью — не проблема: могу закрыть его и оставить ветку просто как тестовую сборку, этот комментарий останется её описанием.

@AgelxNash

Copy link
Copy Markdown
Author

Закрываю PR пообещанным выше способом: ветка integration/open-prs-20260906 в форке остаётся доступной как тестовая сборка (описание и списки PR — в комментарии выше и в журнальном коммите). Если команде будет полезна пересборка после принятия части PR — скажите, соберу заново.

@AgelxNash AgelxNash closed this Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants