Skip to content

I've upgraded the PHP packages and added support for PHP 8.4.#249

Merged
dmulitsa merged 1 commit into
sugarcrm:developfrom
klimslim:php84up
Jul 22, 2025
Merged

I've upgraded the PHP packages and added support for PHP 8.4.#249
dmulitsa merged 1 commit into
sugarcrm:developfrom
klimslim:php84up

Conversation

@klimslim
Copy link
Copy Markdown

Here's a summary of the changes:

  • I updated composer.json to set the PHP requirement to >=8.2, upgraded phpunit/phpunit to ^11.0, and squizlabs/php_codesniffer to ^3.0.
  • I updated .travis.yml to ensure testing against PHP 8.2, 8.3, and 8.4.
  • I ran composer update to refresh the dependencies.
  • I used Rector to refactor the code for PHP 8.4 compatibility.
  • I addressed any failing tests that arose after the upgrade.
  • I ran PHP_CodeSniffer to correct coding style violations.

feat: Migrate CI from Travis CI to GitHub Actions

This commit migrates the continuous integration pipeline from Travis CI to GitHub Actions.

The new GitHub Actions workflow (.github/workflows/ci.yml) includes the following features:

  • Matrix testing for PHP versions 8.2, 8.3, and 8.4.
  • Composer dependency installation and caching.
  • Execution of style checks (composer check-style).
  • Execution of unit tests (composer tests-ci).
  • Uploading of code coverage reports to Coveralls.

The .travis.yml file has been removed.
The README.md has been updated to reflect the change to GitHub Actions, including new build status badges.

Note for repository administrators:
For Coveralls integration to work correctly, ensure that COVERALLS_REPO_TOKEN is not set as a secret if using secrets.GITHUB_TOKEN with the coverallsapp/github-action. The official Coveralls GitHub Action recommends using secrets.GITHUB_TOKEN.

Remove tests cache

@klimslim klimslim force-pushed the php84up branch 4 times, most recently from 5ab1d63 to 02a7b12 Compare June 16, 2025 19:37
Here's a summary of the changes:
- I updated `composer.json` to set the PHP requirement to `>=8.2`, upgraded `phpunit/phpunit` to `^11.0`, and `squizlabs/php_codesniffer` to `^3.0`.
- I updated `.travis.yml` to ensure testing against PHP 8.2, 8.3, and 8.4.
- I ran `composer update` to refresh the dependencies.
- I used Rector to refactor the code for PHP 8.4 compatibility.
- I addressed any failing tests that arose after the upgrade.
- I ran PHP_CodeSniffer to correct coding style violations.

feat: Migrate CI from Travis CI to GitHub Actions

This commit migrates the continuous integration pipeline from Travis CI to GitHub Actions.

The new GitHub Actions workflow (`.github/workflows/ci.yml`) includes the following features:
- Matrix testing for PHP versions 8.2, 8.3, and 8.4.
- Composer dependency installation and caching.
- Execution of style checks (`composer check-style`).
- Execution of unit tests (`composer tests-ci`).
- Uploading of code coverage reports to Coveralls.

The `.travis.yml` file has been removed.
The `README.md` has been updated to reflect the change to GitHub Actions, including new build status badges.

Note for repository administrators:
For Coveralls integration to work correctly, ensure that `COVERALLS_REPO_TOKEN` is not set as a secret if using `secrets.GITHUB_TOKEN` with the `coverallsapp/github-action`. The official Coveralls GitHub Action recommends using `secrets.GITHUB_TOKEN`.
@dmulitsa dmulitsa merged commit f3680d4 into sugarcrm:develop Jul 22, 2025
3 checks passed
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