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

Laravel closure #983

Merged
merged 6 commits into from
Feb 7, 2022
Merged

Laravel closure #983

merged 6 commits into from
Feb 7, 2022

Conversation

veewee
Copy link
Contributor

@veewee veewee commented Feb 7, 2022

Q A
Branch master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Documented? no
Fixed tickets Partially #957

Replaces opis/closure with laravel/serializable-closure for PHP 81 compatibility

@veewee veewee added this to the 1.8.0 milestone Feb 7, 2022
@@ -85,6 +85,8 @@ function it_has_convention_at_another_location_through_cli_params()
/** @test */
function it_has_grumphp_in_root_but_composer_in_project_folder()
{
$this->markTestSkipped('Broken test... Unable to locate autoloader!');
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both skipped tests can't find the custom validation task since the autoloader cannot be found.
This must have been an issue that has been around much longer.

Looking into a way for fixing this, but it can be done later on. Main focus is releasing a compatible grumphp version.

@veewee veewee merged commit ec68194 into phpro:master Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant