PRE-3251: adding CI to generate release in php 7.2#15
PRE-3251: adding CI to generate release in php 7.2#15adumont-payplug merged 3 commits intofeature/PRE-3254from
Conversation
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
2 times, most recently
from
5fe3ddb to
7632499
Compare
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
2 times, most recently
from
cdb4156 to
2191aa5
Compare
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
11 times, most recently
from
9067b30 to
62640e4
Compare
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
from
62640e4 to
19c819e
Compare
There was a problem hiding this comment.
Pull request overview
This PR introduces a CI workflow and supporting scripts/config to produce a PHP 7.2-compatible “release” build by copying the sources to a separate directory, running Rector downgrades, and generating a trimmed composer.json for the packaged artifact.
Changes:
- Add GitHub Actions CI workflow that runs quality checks, downgrades to PHP 7.2, validates, tests, packages artifacts, and attempts to create releases on tags.
- Add a release-only Rector config (
rector-release.php) and a PHP script to generate a PHP 7.2-compatiblecomposer.json. - Add local release automation via
composer.jsonscript and Makefile target; ignore generated output directory.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/generate-release-composer.php |
Generates a release composer.json by modifying the root composer.json for PHP 7.2 compatibility. |
rector.php |
Removes the default Rector config file. |
rector-release.php |
Adds Rector downgrade configuration targeting the generated release directory. |
composer.json |
Adds a release script pipeline (copy → downgrade → generate composer.json). |
Makefile |
Adds a release target and rewires rector-dry. |
.gitignore |
Ignores a generated release directory. |
.github/workflows/ci.yml |
Adds CI jobs for quality, downgrade, validation, testing, packaging, and release-on-tag. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
from
19c819e to
db0df82
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
from
db0df82 to
193615a
Compare
adumont-payplug
force-pushed
the
feature/PRE-3251
branch
from
193615a to
9cf0a1c
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.