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

[1.4] Fix auto-generated return values with union types #1144

Merged
merged 1 commit into from Sep 13, 2021
Merged

[1.4] Fix auto-generated return values with union types #1144

merged 1 commit into from Sep 13, 2021

Conversation

GrahamCampbell
Copy link
Contributor

@GrahamCampbell GrahamCampbell commented Sep 7, 2021

Same as #1143 for the 1.4 branch. Easier than dealing with those merge PRs.


Blocks laravel/framework#38404. cc @davedevelopment, @driesvints.

@driesvints
Copy link

@davedevelopment definitely not want to pressure you (take your time) but this is the last thing we need to provide PHP 8.1 support in Laravel. Would be cool if we could merge and tag this soon! :)

@davedevelopment
Copy link
Collaborator

No worries, will tag shortly

@driesvints
Copy link

Thanks @davedevelopment! Appreciate it 👍

@driesvints
Copy link

I'm sorry @davedevelopment, seems like there are a couple of more snags to be solved before we have PHP 8.1 support in Laravel. But this PR definitely brought us closer so thank you!

@davedevelopment
Copy link
Collaborator

@driesvints no worries, good luck :)

@GrahamCampbell GrahamCampbell deleted the 14-fix-union-return branch September 13, 2021 17:10
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

Successfully merging this pull request may close these issues.

None yet

3 participants