Skip to content

HQD-736: run tests on GitHub Actions with PHP 8.5 - #125

Merged
SilverFire merged 1 commit into
masterfrom
hqd-736-gha-php85
Nov 13, 2026
Merged

SilverFire merged 1 commit into
masterfrom
hqd-736-gha-php85

Conversation

@SilverFire

@SilverFire SilverFire commented Nov 13, 2026 •

Copy link
Copy Markdown
Member
  • Remove .travis.yml (Travis CI no longer used; all CI runs on GitHub Actions).
  • Workflows (phpunit, behat, psalm, phpcs) already run on PHP 8.5 only; make composer install run on every build instead of being skipped on a vendor/ cache hit keyed on composer.json alone, so dev-master dependencies are never stale.

Summary by CodeRabbit

  • Chores
    • Composer dependencies are now installed consistently across automated test and code-quality checks, including when a cache is available.
    • Removed the legacy continuous-integration configuration.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

Four GitHub Actions workflows now run Composer installation whether or not the Composer cache is hit. The Travis CI configuration was removed.

Changes: CI configuration

Layer / File(s) Summary
Unconditional Composer installation
.github/workflows/behat-tests.yml, .github/workflows/phpcs.yml, .github/workflows/phpunit-tests.yml, .github/workflows/psalm-analysis.yml
Each workflow now runs Composer installation regardless of Composer cache status.
Travis CI configuration removal
.travis.yml
The Travis CI configuration and its PHP versions, environment, cache settings, lifecycle commands, and matrix setting were removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: hiqsol, vadymhrechukha

Merge Risk: Minimal · up to dd8f6

This change only updates CI configuration. Dependencies are now installed on every workflow run, and the unused Travis CI configuration is removed. No application behavior changes, and merge risk is minimal.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "HQD-736: run tests on GitHub Actions with PHP 8.5" describes a change the objectives do not support. The PR removes .travis.yml, makes composer install run on every CI build, and aims to pr…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "HQD-736: run tests on GitHub Actions with PHP 8.5" describes a change the objectives do not support. The PR removes .travis.yml, makes composer install run on every CI build, and aims to prevent stale dev-master dependencies. The objectives say the PHP 8.5 workflows already exist, and the commit message describes the change as CI work for GitHub Actions with PHP 8.5. The title names the PHP 8.5 and GitHub Actions work but omits the main change, which is removing Travis CI and forcing composer install on each run. It is partially related to the changeset.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d382f481-6b82-492b-8178-ddbf689bbc1d
Commits

Reviewing files that changed from the base of the PR and between 4fa2191 and dd8f60a.

Files selected for processing: 5

  • .github/workflows/behat-tests.yml
  • .github/workflows/phpcs.yml
  • .github/workflows/phpunit-tests.yml
  • .github/workflows/psalm-analysis.yml
  • .travis.yml
Files with no reviewable changes: 5

  • .github/workflows/behat-tests.yml
  • .github/workflows/phpcs.yml
  • .github/workflows/phpunit-tests.yml
  • .github/workflows/psalm-analysis.yml
  • .travis.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@SilverFire SilverFire closed this Nov 13, 2026
@SilverFire SilverFire reopened this Nov 13, 2026
@SilverFire
SilverFire merged commit 91fa514 into master Nov 13, 2026
5 checks passed
@SilverFire
SilverFire deleted the hqd-736-gha-php85 branch November 13, 2026 14:00
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.

1 participant