Skip to content

Laravel 13.x Compatibility#117

Open
laravel-shift wants to merge 2 commits intoRickDBCN:2.xfrom
laravel-shift:l13-compatibility
Open

Laravel 13.x Compatibility#117
laravel-shift wants to merge 2 commits intoRickDBCN:2.xfrom
laravel-shift:l13-compatibility

Conversation

@laravel-shift
Copy link
Copy Markdown

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 13.x.

Before merging, you need to:

  • Checkout the l13-compatibility branch
  • Review all comments for additional changes
  • Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

@laravel-shift
Copy link
Copy Markdown
Author

⚠️ Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

@laravel-shift
Copy link
Copy Markdown
Author

⚠️ Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 13. However, you should review these changes to ensure the desired combination of versions are built for your package.

@laravel-shift
Copy link
Copy Markdown
Author

⚠️ Laravel 13 now throws a LogicException if you attempt to create new instances or register observer events while booting models. Instead, you must wrap this code within a whenBooted callback.

For example:

static::whenBooted(fn () => static::observe(new ModelObserver));

Shift found boot methods within the following files. You should review the code within these methods and update it to use the whenBooted callback if necessary.

@laravel-shift
Copy link
Copy Markdown
Author

⚗️ Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.

To do so, temporarily add Shift's fork to the repositories property of your composer.json:

{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/filament-email.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "rickdbcn/filament-email": "dev-l13-compatibility",
    }
}

Finally, run: composer update

@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Mar 5, 2026

PR Summary 📝

  • Better Workflow for a Newer Version - Laravel 13
    Hey, our GitHub Actions workflow has been tweaked to work even smoother with Laravel 13 (the latest and the greatest version of Laravel). This means our project can take full advantage and continue to improve! 🙌

  • Updates to run-tests.yml
    There's also another goodie: adjustments have been made to the run-tests.yml file to include compatibility with Laravel 13. This ensures all our testings set up will be working well with this new version! 🏃💨

  • Enhancements to composer.json
    We also polished up our composer.json file, adding support for Laravel 13 under both "require" and "require-dev" parts. This allows us to roll with Laravel 13, whether in production or during development/testing. 🧰🚀

Can't wait to see how these updates bring our project up to speed with all that Laravel 13 has to offer! 🥳🔧

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