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

TASK: Declare compatible with Neos 8 and Flow 8 #27

Merged
merged 2 commits into from
Oct 26, 2023

Conversation

kdambekalns
Copy link
Contributor

This drops compatibility with Neos < 5.3 / Flow < 6.3 and raises the
minimum PHP version to 7.4

This drops compatibility with Neos < 5.3 / Flow < 6.3 and raises the
minimum PHP version to 7.4
@kdambekalns
Copy link
Contributor Author

Note: this won't work on Neos 8 since there is no released version of heise/shariff yet with support for PHP 8.

A PR exists (heiseonline/shariff-backend-php#181) but that needs to be pulled in manually for now…

@dlubitz
Copy link

dlubitz commented Jun 22, 2023

Shariff has released the 10.0 version with PHP 8.1 compatibility.
https://github.com/erkenes/shariff-backend-php/releases/tag/10.0.0

This enables support for PHP 8, and does not contain any breaking changes  for us according to the changelog.
@nilsjung
Copy link
Contributor

nilsjung commented Oct 5, 2023

Sorry to disappoint but if I'm correct erkenes/heise-shariff has released v10 and not heise/shariff.

We internally use also some hacky solutions to bypass the missing heise/shariff update. But maybe its time to use erkenes/heise-shariff as dependency here.

@nilsjung nilsjung merged commit 9e27eea into networkteam:master Oct 26, 2023
@nilsjung
Copy link
Contributor

Heise has released v10, thus, I will finally merge this PR

https://github.com/heiseonline/shariff-backend-php/releases/tag/v10.0.0

@kdambekalns kdambekalns deleted the task/neos-and-php-8 branch October 26, 2023 07:27
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