search

laminas-api-tools / api-tools-admin-ui

Laminas API Tools Admin UI module
https://api-tools.getlaminas.org/documentation
BSD 3-Clause "New" or "Revised" License
6 stars 11 forks source link

Drop PHP 7 and add PHP 8.2 Support #31

Closed rogervila closed 1 year ago

rogervila commented 1 year ago
Q A
Documentation no
Bugfix no
BC Break no
New Feature no
RFC no
QA no

Description

Drop PHP 7 and add PHP 8.2 Support

rogervila commented 1 year ago

Hi @Ocramius, who could review this PR? Thank you!

Ocramius commented 1 year ago

Hmm, no QA checks did run here 🤔

rogervila commented 1 year ago

Hmm, no QA checks did run here 🤔

There are no PHP tests on the package, so maybe laminas/laminas-continuous-integration-action does not apply here.

I propose changing the GitHub actions config to use NodeJS to run JavaScript tests.

Ocramius commented 1 year ago

Ah, true that!

No, all good if the PHP side does nothing 👍

Ocramius commented 1 year ago

Thanks @rogervila!