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

Upgrade to php 8.0.24 and fuelphp 1.9-develop #1407

Merged
merged 3 commits into from
Oct 31, 2022

Conversation

iturgeon
Copy link
Member

@iturgeon iturgeon commented Oct 19, 2022

PHP support for 7 is ending very soon, so this is an attempt to update to give the team some more runway. PHP 8.0.x doesn't have a very long life either, but this is better than nothing.

Upgrading to PHP 8.0.x required upgrading fuel to a development version unfortunately - there's no official non-dev release that has the updates required to work with PHP 8.0.x

PHP support for 7 is ending very soon, so this is
and attempt to update to give the team some more runway.
PHP 8.0.x doesn't have a very long life either, but this
is better than nothing.

Upgrading to PHP 8.0.x required upgrading fuel to a development
version unfortunately - there's no official full release that has
the updates required to work with PHP 8.0.x
@iturgeon iturgeon mentioned this pull request Oct 19, 2022
@clpetersonucf
Copy link
Member

The changes look good, just need you to update the target branch to dev/9.0.2. Our fork of the fuelphp codesniffer is woefully out of date too, and appears to be choking on now-deprecated curly brace syntax for arrays.

@iturgeon iturgeon changed the base branch from master to dev/9.0.2 October 20, 2022 18:27
@iturgeon
Copy link
Member Author

iturgeon commented Oct 20, 2022

@clpetersonucf I re-targeted the desired branch and have a pull request open over at ucfcdl/fuelphp-phpcs#1 to fix the code sniffer so our ci_tests will pass on Materia. It'll require a few steps back and forth to get it working. (first the pr gets merged, then tagged, then the composer.json and .lock file need updates here in Materia)

@clpetersonucf
Copy link
Member

ucfcdl/fuelphp-phpcs updated and tagged to v3.0.1.

@clpetersonucf clpetersonucf added the Awaiting Additional Updates This PR was reviewed, and is awaiting updates label Oct 24, 2022
@clpetersonucf clpetersonucf merged commit 29a4d9e into ucfopen:dev/9.0.2 Oct 31, 2022
@clpetersonucf clpetersonucf added hacktoberfest-accepted and removed Awaiting Additional Updates This PR was reviewed, and is awaiting updates labels Oct 31, 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.

None yet

2 participants