Remove L10 from test matrix

[PeteBishwhip]

This pull request includes a change to the GitHub Actions workflow configuration to update the test matrix for Laravel versions.

Changes in .github/workflows/run-tests.yml:

• Removed support for Laravel 10.x from the test matrix.

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

[SRWieZ]

Not important but something to delete here too :
https://github.com/NativePHP/laravel/blob/91f487cb40fdea5ce737fc7128f4f17dc2a2e92f/src/NativeServiceProvider.php#L53C2-L56C12

[PeteBishwhip]

Not important but something to delete here too : https://github.com/NativePHP/laravel/blob/91f487cb40fdea5ce737fc7128f4f17dc2a2e92f/src/NativeServiceProvider.php#L53C2-L56C12

I left that in specifically because it also references L11. Do we not require it for 11?

[SRWieZ]

The comment meant: we have to add the phpstan-ignore because this is backward compatible with L10.

I think the constructor did not have any parameters in L10 if I recall correctly.

[simonhamp]

That FreshCommand is ours tho... why would we need to get phpstan to ignore? If we change the signature, we change the usages, no?

[PeteBishwhip]

Lets check it. I'm going to remove the comment.

[SRWieZ]

I don't recall exactly.

I remember putting this in because there was an error with PHPStan in L10, but L10 was working anyway.

[PeteBishwhip]

@SRWieZ @simonhamp

Whats the use of this constructor argument?

My IDE is reporting its useless and following the trail back, I can't see where it's ever being used? Can this not be updated to not require a constructor arg?

[simonhamp]

L11 FreshCommand is the superclass of our FreshCommand

[simonhamp]

L10 didn't have a constructor. So it looks like the PHPStan comment was to prevent L10 warnings.

I think we can remove the comment now. But we should leave the argument in place, as this will be required. Not sure why your IDE isn't picking it up...

[PeteBishwhip]

Hah! Dev dependency issues! I see it now. Thanks.

[SRWieZ]

All green