Skip to content

feat: add Laravel 13 support #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
shavonn merged 1 commit into from
Mar 31, 2026
Merged

feat: add Laravel 13 support #10

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,11 @@ jobs:
fail-fast: true
matrix:
php: [ 8.2, 8.3, 8.4 ]
laravel: [ 11.*, 12.* ]
laravel: [ 11.*, 12.*, 13.* ]
stability: [ prefer-lowest, prefer-stable ]
exclude:
- php: 8.2
laravel: 13.*

name: PHP ${{ matrix.php }} - Laravel ${{ matrix.laravel }} - ${{ matrix.stability }}

Expand Down
7 changes: 4 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ records, with no user resolution or driver complexity.
## Requirements

- PHP 8.2+
- Laravel 11 or 12
- Laravel 11 or 12 or 13
Comment on lines 28 to +31

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

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

The Requirements section now lists Laravel 13 support, but it still states "PHP 8.2+" without clarifying that Laravel 13 is not compatible with PHP 8.2 (the CI matrix explicitly excludes PHP 8.2 + Laravel 13). Consider updating the requirement wording to reflect the effective minimum PHP version per Laravel major (e.g., PHP 8.2+ for Laravel 11/12, PHP 8.3+ for Laravel 13) so users aren’t misled.

Copilot uses AI. Check for mistakes.

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

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

Minor wording: "Laravel 11 or 12 or 13" reads awkwardly in documentation. Consider rephrasing to "Laravel 11, 12, or 13" (or "Laravel 11–13") for clarity.

Suggested change
- Laravel 11 or 12 or 13
- Laravel 11, 12, or 13

Copilot uses AI. Check for mistakes.

## Installation

Expand Down Expand Up @@ -188,7 +188,8 @@ The model automatically clears the cache when toggles are saved or deleted.

### Inertia

Share all toggles with your frontend by using the provided Inertia middleware. Replace your `HandleInertiaRequests` middleware in `bootstrap/app.php`:
Share all toggles with your frontend by using the provided Inertia middleware. Replace your `HandleInertiaRequests`
middleware in `bootstrap/app.php`:

```php
use OffloadProject\Toggle\Middleware\ShareTogglesWithInertia;
Expand All @@ -204,7 +205,7 @@ All toggles will be available as the `flags` prop in your frontend:

```js
// Vue/React
const { flags } = usePage().props
const {flags} = usePage().props

if (flags.newCheckout) {
// Show new checkout
Expand Down
8 changes: 4 additions & 4 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,16 +17,16 @@
],
"require": {
"php": "^8.2",
"illuminate/contracts": "^11.0|^12.0",
"illuminate/database": "^11.0|^12.0",
"illuminate/support": "^11.0|^12.0"
"illuminate/contracts": "^11.0|^12.0|^13.0",
"illuminate/database": "^11.0|^12.0|^13.0",
"illuminate/support": "^11.0|^12.0|^13.0"
},
"require-dev": {
"captainhook/captainhook-phar": "^5.27",
"inertiajs/inertia-laravel": "^2.0",
"larastan/larastan": "^3.8.1",
"laravel/pint": "^1.26.0",
"orchestra/testbench": "^9.0|^10.8.0",
"orchestra/testbench": "^9.0|^10.8|^11.0",
"pestphp/pest": "^3.0|^4.0",
"pestphp/pest-plugin-laravel": "^3.0|^4.0",
"ramsey/conventional-commits": "^1.6"
Expand Down
Loading