Skip to content
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.

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

Method signature issues in LighthouseServiceProvider#loadRoutesFrom #1590

Closed
voydz opened this issue Oct 20, 2020 · 3 comments
Closed

Method signature issues in LighthouseServiceProvider#loadRoutesFrom #1590

voydz opened this issue Oct 20, 2020 · 3 comments

Comments

@voydz
Copy link

voydz commented Oct 20, 2020

Describe the bug

Lighthouse is not able to fire up its service provider in Laravel 8.11.x due to laravel/framework#34866

For example running php artisan results in an ErrorException with message Declaration of Nuwave\Lighthouse\LighthouseServiceProvider::loadRoutesFrom($path): void should be compatible with Illuminate\Support\ServiceProvider::loadRoutesFrom($path, array $attributes = Array)

Expected behavior/Solution

Running php artisan or any other way of booting a Laravel application running fine.

Steps to reproduce

  1. Install Laravel 8.11.x
  2. php artisan
  3. get an ErrorException

Output/Logs

Click to expand
 ErrorException 

  Declaration of Nuwave\Lighthouse\LighthouseServiceProvider::loadRoutesFrom($path): void should be compatible with Illuminate\Support\ServiceProvider::loadRoutesFrom($path, array $attributes = Array)

  at vendor/nuwave/lighthouse/src/LighthouseServiceProvider.php:193

Lighthouse Version

using the latest version 4.17.0

Note: PR is incomming.

best regards

@voydz
Copy link
Author

voydz commented Oct 20, 2020

It seems the change was reverted in laravel/framework@7fff6ea.

@spawnia
Copy link
Collaborator

spawnia commented Oct 21, 2020

@voydz thanks for looking into this and notifying the Laravel team about the breakage. Looks like we are good for now, let's close the issue then?

@voydz
Copy link
Author

voydz commented Oct 21, 2020

@spawnia Sounds fine to me ;)

@voydz voydz closed this as completed Oct 21, 2020
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

No branches or pull requests

2 participants