Skip to content

Laravel 13.x Compatibility#12

Open
laravel-shift wants to merge 1 commit intoopentelemetry-php:mainfrom
laravel-shift:l13-compatibility
Open

Laravel 13.x Compatibility#12
laravel-shift wants to merge 1 commit intoopentelemetry-php:mainfrom
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

⚠️ The latest version of Laravel has a dependency for phpunit/phpunit of ^12.5. If you would like to also upgrade your tests, you may run the PHPUnit 12 Shift for free.

@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

⚗️ 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/contrib-auto-laravel.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "open-telemetry/opentelemetry-auto-laravel": "dev-l13-compatibility",
    }
}

Finally, run: composer update

@ivanvasechko
Copy link
Copy Markdown
Contributor

ivanvasechko commented Mar 18, 2026

Hi @ChrisLightfootWild, do you think we can get it merged?

Actually, I think it should be done as part of https://github.com/open-telemetry/opentelemetry-php-contrib/pull/525/changes

@puremana
Copy link
Copy Markdown

Would be awesome to get these changes in an upcoming version in the next couple of weeks 😃

@ChrisLightfootWild
Copy link
Copy Markdown
Contributor

This is being tracked in open-telemetry/opentelemetry-php-contrib#525.

@krisimmig
Copy link
Copy Markdown

This is being tracked in open-telemetry/opentelemetry-php-contrib#525.

The referenced MR is merged.

Looks like this MR is adding later versions to some allowed packages and also re-formats the composer.json file to 4 spaces for tabs. Can we merge this? Happy to help if needed :)

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.

5 participants