Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

To support php 8.2 #114

Closed
jskorlol opened this issue Jan 25, 2023 · 1 comment · Fixed by #115
Closed

To support php 8.2 #114

jskorlol opened this issue Jan 25, 2023 · 1 comment · Fixed by #115
Assignees
@tarampampam
Labels
type:enhancement New feature or request

Comments

@jskorlol
Copy link

jskorlol commented Jan 25, 2023 •
edited
Loading

I have a problem using it in php 8.2.

Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/vendor/spiral/roadrunner-laravel/src/Console/Commands/WorkerFactory.php on line 59
Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/vendor/spiral/roadrunner-laravel/src/Console/Commands/WorkerFactory.php on line 63

https://github.com/spiral/roadrunner-laravel/blob/master/src/Console/Commands/WorkerFactory.php#L59
https://github.com/spiral/roadrunner-laravel/blob/master/src/Console/Commands/WorkerFactory.php#L63

https://php.watch/versions/8.2/$%7Bvar%7D-string-interpolation-deprecated

I have that problem.

Any chance of supporting PHP 8.2?
@jskorlol jskorlol added the type:enhancement New feature or request label Jan 25, 2023
@tarampampam
Copy link
Collaborator

Yes, sure! I will make a fix tomorrow, okay?

@tarampampam tarampampam mentioned this issue Jan 26, 2023
Merged
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tarampampam tarampampam

Labels
type:enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Strings interpolation for PHP 8.2 roadrunner-php/laravel-bridge
2 participants
@tarampampam @jskorlol