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

PHP8.2 Support in CMS5 for core modules #10403

Closed
2 of 3 tasks
maxime-rainville opened this issue Jul 8, 2022 · 3 comments
Closed
2 of 3 tasks

PHP8.2 Support in CMS5 for core modules #10403

maxime-rainville opened this issue Jul 8, 2022 · 3 comments

Comments

@maxime-rainville
Copy link
Contributor

maxime-rainville commented Jul 8, 2022

The first alpha of PHP8.2 is available. It's meant to reach feature freeze on July 19.

Acceptance criteria

  • We have a clearly documented way of running a dev environment on pre-release PHP8.2. That's mostly targeted toward the internal SIlverstripe dev team. You should be able to install PHP 8.2 just like any other version as it's now stable
  • CMS5 core and supported module builds can run PHP8.2. << I've split off a [new issue for supported modules](PHP 8.2 Support in CMS5 for non-core modules #10618)
  • Syntax/constructs deprecated in PHP8.2 are no longer used.

Notes

New issues created

Third party issues created

Third party PRs

CI Integration test

- https://github.com/emteknetnz/silverstripe-installer/actions/runs/3711071825

PRs

@emteknetnz
Copy link
Member

emteknetnz commented Dec 14, 2022

We're blocked on:

  • webonyx/graphql-php v15 being tagged stable / us upgrading silverstripe/graphl to use v15
  • oscarotero/html-parser PR being merged and tagged
  • CMS 5 compatible kitchen-sink branches being available

I've reduced the scope of this issue to "core" modules only and I've created a new issue to handle kitchen-sink

Given that graphql is "core", even when we close this issue we still won't have core support for PHP 8.2 - though we should once we've completed the webonyx/graphql-php v15 issue

@GuySartorelli
Copy link
Member

GuySartorelli commented Dec 21, 2022

@emteknetnz Your external PR oscarotero/html-parser#4 has been merged and tagged - do we need to change the minimum depencency or are we okay with --prefer-lowest not being 8.2 compatible? I think just leave as is but wanna confirm.

If we don't need to change that, this can be closed.

@emteknetnz
Copy link
Member

May as well update the minimum installable version #10628

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants