Skip to content

Add Statamic 6 / PHP 8.4 / Laravel 12 to test matrix #57

@robocopklaus

Description

@robocopklaus

Summary

The CI test matrix only covers Statamic 5 / PHP 8.3 / Laravel 11, but composer.json declares "statamic/cms": "^5.0|^6.0". Statamic 6 compatibility is claimed but not CI-tested.

Problem

# .github/workflows/tests.yml — current matrix
strategy:
  matrix:
    include:
      - php: 8.3
        laravel: 11
        statamic: 5
        testbench: 9

Only one matrix entry exists. There is no testing against:

  • Statamic 6
  • PHP 8.4
  • Laravel 12
  • Orchestra Testbench 10

Impact

Breaking changes in Statamic 6 or Laravel 12 would not be caught by CI. Users upgrading to Statamic 6 could encounter runtime errors.

Suggested Fix

strategy:
  matrix:
    include:
      - php: 8.3
        laravel: 11
        statamic: 5
        testbench: 9
      - php: 8.4
        laravel: 12
        statamic: 6
        testbench: 10

Also update require-dev to allow broader ranges:

"orchestra/testbench": "^9.0|^10.0",
"pestphp/pest": "^2.34|^3.0"

References

  • File: .github/workflows/tests.yml:17-24
  • File: composer.json
  • Consolidated review finding: V6

Metadata

Metadata

Assignees

No one assigned

    Labels

    github_actionsPull requests that update GitHub Actions codehighHigh severity finding

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions