Skip to content

Conversation

CodeExplorerX
Copy link

Update php to version 8.2

@CodeExplorerX CodeExplorerX deleted the patch-1 branch October 22, 2023 09:55
@CodeExplorerX CodeExplorerX restored the patch-1 branch October 22, 2023 09:56
@CodeExplorerX CodeExplorerX reopened this Oct 22, 2023
Copy link
Author

@CodeExplorerX CodeExplorerX left a comment

Choose a reason for hiding this comment

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

Update php to version 8.2

@QuintenQVD0
Copy link
Contributor

The panel does not run on PHP 8.2? aleast the queue worker does not

@CodeExplorerX
Copy link
Author

The panel does not run on PHP 8.2? aleast the queue worker does not

I tested it locally on Debian 12 and Ubuntu 23.04. Pteodactyl panel runs on php 8.2 as well as a queue worker

@QuintenQVD0
Copy link
Contributor

The panel does not run on PHP 8.2? aleast the queue worker does not

I tested it locally on Debian 12 and Ubuntu 23.04. Pteodactyl panel runs on php 8.2 as well as a queue worker

Ubuntu 23.04 is not even an LTS so that to begin with is not supported.

The queue worker does not work last time we checked. It does not run its crons that the users in the panel setup

@QuintenQVD0
Copy link
Contributor

and there are way more places in the docs where php 8.1 is mentioned as we lock everything to a specific version

@CodeExplorerX
Copy link
Author

and there are way more places in the docs where php 8.1 is mentioned as we lock everything to a specific version

Okay, I'll try to finalize and attach all the tests

@Boy132
Copy link
Contributor

Boy132 commented Oct 22, 2023

The panel does not run on PHP 8.2? aleast the queue worker does not

Since the Laravel 10 update it runs fine on php 8.2. The gh tests even use 8.1 & 8.2.

But yes, there are more places in the docs that need to be updated.

@QuintenQVD0
Copy link
Contributor

Please also see #558 and make this pr work with that.

CodeExplorerX

This comment was marked as resolved.

@CodeExplorerX CodeExplorerX closed this by deleting the head repository Mar 2, 2024
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.

3 participants