Skip to content

Fix/linting 2026 01#52

Merged
dermatz merged 6 commits into
developfrom
fix/linting-2026-01
Jan 8, 2026
Merged

Fix/linting 2026 01#52
dermatz merged 6 commits into
developfrom
fix/linting-2026-01

Conversation

@dermatz
Copy link
Copy Markdown
Member

@dermatz dermatz commented Jan 8, 2026

This pull request introduces a comprehensive DDEV-based local development environment for MageForge, including full Magento 2 setup automation, OpenSearch integration, and code quality tooling. It adds custom DDEV commands for installing Magento, running Magento CLI commands, and executing Magento Coding Standard checks. The PR also documents the project architecture, coding conventions, and development workflow in detail.

Environment & Tooling Setup:

  • Added DDEV configuration for Magento 2 development:

    • .ddev/config.yaml: Defines project as Magento 2, sets PHP 8.3, MariaDB 10.6, nginx-fpm, and other project-specific settings.
    • .ddev/addon-metadata/ddev-opensearch/manifest.yaml, .ddev/docker-compose.opensearch.yaml, .ddev/opensearch/Dockerfile: Adds OpenSearch and OpenSearch Dashboards as DDEV services with custom plugins, enabling Magento's OpenSearch support. [1] [2] [3]
  • Custom DDEV commands for Magento workflow:

Documentation & Standards:

  • Project documentation and coding standards:
    • .github/copilot-instructions.md: Thorough instructions on project structure, DDEV workflow, PHP conventions (strict types, property promotion, type hints, enums), builder pattern usage, CLI command structure, linting strategy, git workflow, and testing procedures.

CI Improvements:

@dermatz dermatz requested a review from Morgy93 January 8, 2026 07:51
@dermatz dermatz merged commit f9d0224 into develop Jan 8, 2026
1 of 2 checks passed
@dermatz dermatz deleted the fix/linting-2026-01 branch January 8, 2026 08:08
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