Skip to content

Conversation

@damienlagae
Copy link
Member

No description provided.

@damienlagae damienlagae requested a review from Copilot October 10, 2025 13:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive development infrastructure including documentation, CI/CD workflows, and test coverage to the Enabel UX bundle. The changes establish proper project setup with testing, code quality tools, and developer documentation.

Key changes:

  • Added PHPUnit test suite with coverage for Modal components and bundle functionality
  • Implemented CI/CD pipeline with GitHub Actions for testing, code standards, and static analysis
  • Added development tooling with PHPStan and PHP-CS-Fixer configurations

Reviewed Changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
File Description
composer.json Updated dependencies, autoloading, and added development scripts
README.md Added project documentation with installation and usage instructions
phpunit.xml.dist PHPUnit configuration for test execution and coverage
tests/Helper/ModalTest.php Comprehensive test coverage for Modal helper functionality
tests/EnabelUxBundleTest.php Bundle instantiation and path validation tests
tests/Component/ModalTest.php Basic Modal component test
.github/workflows/ci.yaml CI/CD workflow using reusable Enabel workflows
.php-cs-fixer.dist.php Code formatting standards configuration
phpstan.neon.dist Static analysis configuration
tools/ Development tool configurations for PHPStan and PHP-CS-Fixer
src/ Added license headers to existing source files
docs/modal.md Cleaned up modal documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@damienlagae damienlagae merged commit 60fac0f into main Oct 10, 2025
7 of 8 checks passed
@damienlagae damienlagae deleted the feat/docAndCo branch October 10, 2025 13:18
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