Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Feb 21, 2023

Drop 7.4 and add 8.2

For nextcloud/server#34692

@come-nc come-nc added this to the Nextcloud 26 milestone Feb 21, 2023
@come-nc come-nc self-assigned this Feb 21, 2023
@come-nc
Copy link
Contributor Author

come-nc commented Feb 21, 2023

Who knows enough about centos to update https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/example_centos.rst ?

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

😎

@ChristophWurst
Copy link
Member

Who knows enough about centos to update https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/example_centos.rst ?

@AndrewwHummer @hanserasmus would anyone be able to help us out here?

@MichaIng
Copy link
Member

Would it be probably better to use PHP 8.2 in examples, just so that it does not need to be changed again when Nextcloud drops support for PHP 8.0? 🙂

@come-nc
Copy link
Contributor Author

come-nc commented Feb 21, 2023

Would it be probably better to use PHP 8.2 in examples, just so that it does not need to be changed again when Nextcloud drops support for PHP 8.0? slightly_smiling_face

Could have, should have, will do next time.

I’m merging this, centos thing can be done in an other PR.

@come-nc come-nc merged commit 8c66301 into master Feb 21, 2023
@come-nc come-nc deleted the fix/update-php-requirements branch February 21, 2023 15:50
@ChristophWurst
Copy link
Member

I’m merging this, centos thing can be done in an other PR.

Open a ticket an assign 26 just so this isn't forgotten? thx

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.

5 participants