Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

Laravel 10.x Compatibility #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laravel-shift
Copy link

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

Before merging, you need to:

  • Checkout the l10-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
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
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/active.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "hieu-le/active": "dev-l10-compatibility",
    }
}

Finally, run: composer update

@joshuadwire
Copy link

I can confirm that this library works with just the dependency change included in this PR.

@sebastienheyd
Copy link
Contributor

I also confirm that this can be merged to add compatibility with Laravel 10.

Is this project still active?

I use this package as a dependency of my own packages, it's the last one that is not compatible with Laravel 10.

If the project is no longer being followed I might as well switch to a fork but I'd rather be sure it's not before doing so.

@Briareos17
Copy link

@letrunghieu Merge please

@lloricode
Copy link
Contributor

this must be abandon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants