Skip to content

feat!: drop Laravel 11 support#104

Open
gjtorikian wants to merge 1 commit into
mainfrom
deps/drop-laravel-11
Open

feat!: drop Laravel 11 support#104
gjtorikian wants to merge 1 commit into
mainfrom
deps/drop-laravel-11

Conversation

@gjtorikian

Copy link
Copy Markdown
Contributor

Laravel 11 has unpatched security issues that are not going to get fixed because the framework is EOL. This PR drops support for it in this package.

@gjtorikian gjtorikian requested review from a team as code owners June 17, 2026 17:14
@gjtorikian gjtorikian requested a review from nicknisi June 17, 2026 17:14
@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown

Greptile Summary

This PR drops Laravel 11 support from the package, motivated by unpatched security vulnerabilities in the now-EOL framework version.

  • composer.json: Removes the ^11.0 constraint from illuminate/contracts and illuminate/support, drops orchestra/testbench ^9.0 (the Laravel 11 testbench series), and removes the "laravel 11" keyword.
  • .github/workflows/ci.yml: Removes '11.*' from the Laravel test matrix; the remaining PHP 8.2 × Laravel 13.* exclusion is still accurate.

Confidence Score: 5/5

All changes are consistent and correctly scoped to removing Laravel 11 — safe to merge.

The dependency constraints, testbench version, CI matrix, and keywords are all updated in lockstep. No remaining references to Laravel 11 were left behind, and the existing PHP 8.2 × Laravel 13 exclusion in the CI matrix is still correct.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/ci.yml Removes '11.' from the Laravel matrix; remaining exclusion (PHP 8.2 × Laravel 13.) is still valid.
composer.json Drops illuminate/contracts and illuminate/support ^11, removes orchestra/testbench ^9.0 (Laravel 11 testbench), and cleans the "laravel 11" keyword — all consistently aligned with the Laravel 11 drop.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph CI Matrix ["CI Test Matrix (after PR)"]
        P82["PHP 8.2"] --> L12["Laravel 12.*"]
        P83["PHP 8.3"] --> L12
        P83 --> L13["Laravel 13.*"]
        P84["PHP 8.4"] --> L12
        P84 --> L13
        P82 -. excluded .-> L13
    end
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    subgraph CI Matrix ["CI Test Matrix (after PR)"]
        P82["PHP 8.2"] --> L12["Laravel 12.*"]
        P83["PHP 8.3"] --> L12
        P83 --> L13["Laravel 13.*"]
        P84["PHP 8.4"] --> L12
        P84 --> L13
        P82 -. excluded .-> L13
    end
Loading

Reviews (1): Last reviewed commit: "feat!: drop Laravel 11 support" | Re-trigger Greptile

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant