-
-
Notifications
You must be signed in to change notification settings - Fork 116
New GH Workflow #629
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
New GH Workflow #629
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deleted outdated config files (ecs.php, deptrac.yaml, rector.php, infection.json, etc.), old workflows, and replaced `autoload.php` with a centralized `bootstrap.php`. Introduced a streamlined CI workflow with `.github/workflows/ci.yml`.
Added detailed help descriptions to all Symfony Console commands, replacing older `setHelp()` calls. Removed obsolete test dependencies and configurations for better maintainability.
Updated the workflow condition for mutation testing jobs to trigger only on branches matching the `X.X.x` pattern. Removed redundant build workflows to streamline configuration.
Standardized command attributes formatting across multiple files for improved readability. Cleaned up unused imports in test classes and ensured consistent order. Simplified command execution parameter formatting in test cases.
…nheritance rules Revised baseline to reflect updated checks on constructor parameter defaults, nullable declarations, and inheritance restrictions. Removed obsolete error messages for better alignment with current codebase.
…or branch pattern condition
Simplified the branch condition for mutation testing jobs by replacing `matches` with `endsWith`.
Removed the `docker_lint` job from the CI workflow for simplification. Adjusted dependency for `prepare_dependencies` job accordingly.
Standardized indentation and file-specific rules in `.editorconfig`. Adjusted export-ignore rules in `.gitattributes` to minimize unnecessary exclusions and improve maintainability. Updated namespace in `ComposerJsonTest.php`.
Updated the `EXPECTED` files list in the CI workflow to reflect the revised export requirements. Removed the unnecessary PHPStan ignore comment in `CallableChecker` as it's no longer relevant.
Enhanced the `tests` job to include a PHP version matrix (8.2-8.5) for broader compatibility testing. Updated Docker image reference to dynamically use the appropriate version. Added a pre-check step to verify the availability of Docker images and conditionally run PHPUnit based on the result. Disabled fail-fast for improved matrix execution.
Removed PHP 8.5 from the version matrix to align with supported versions. Simplified PHPUnit execution by removing the pre-check for Docker images and its related conditions.
Introduced `lowest-deps` testing for PHP 8.2 to ensure compatibility with minimum dependency versions. Adjusted cache key structure to include PHP version and dependency type for improved cache management. Simplified dependency installation logic in tests job.
…dynamically in Docker image Added `PHP_VERSION` as an environment variable in the workflow matrix and updated `castor.php` to dynamically use this variable for Docker image references. Default PHP version fallback logic added.
Removed `actions/cache` steps for dependencies and PHPUnit results/coverage in the `tests` and `infection` jobs to simplify the workflow and reduce potential cache-related issues.
Removed `lowest-deps` testing for PHP 8.2 in the workflow matrix. Updated the `install` task in `castor.php` to support installing dependencies with the `--prefer-lowest` flag when required.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Target branch: 4.1.x
Resolves issue #
Includes: