daniel-de-wit / lighthouse-sanctum

Laravel Sanctum support for Laravel Lighthouse
MIT License
57 stars 9 forks source link

Laravel 11.x Compatibility #99

Closed laravel-shift closed 5 months ago

laravel-shift commented 7 months ago

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

Before merging, you need to:

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

laravel-shift commented 7 months ago

:alembic: 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/lighthouse-sanctum.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "daniel-de-wit/lighthouse-sanctum": "dev-l11-compatibility",
    }
}

Finally, run: composer update

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8121236181

Details


Totals Coverage Status
Change from base Build 4923888399: 0.0%
Covered Lines: 269
Relevant Lines: 270

💛 - Coveralls
eshimischi commented 6 months ago

Lighthouse PR is not migrated yet

amharris commented 5 months ago

Lighthouse PR is not migrated yet

The above/referenced PR for Lighthouse to support Laravel 11 has now been merged.

From initial testing, running this dev branch with Laravel 11 seems to be okay, and no glaring issues have occurred so far. Probably safe to merge.

ibrahem-kamal commented 5 months ago

@daniel-de-wit any estimation on this PR ?

daniel-de-wit commented 5 months ago

@ibrahem-kamal @amharris Sorry for the delay, I was waiting for Lighthouse release indeed.

amharris commented 5 months ago

Thanks, @daniel-de-wit