search

mezzio / mezzio-skeleton

Laminas mezzio skeleton. Begin developing PSR-15 middleware applications in seconds!
https://docs.mezzio.dev/mezzio/
BSD 3-Clause "New" or "Revised" License
115 stars 31 forks source link

PHP 8.1 Support #59

Closed lukaslangen closed 2 years ago

lukaslangen commented 2 years ago

Bug Report

Q A
Version(s) 3.11.x

Summary

Although the project's composer.json states, that php 8.1 is supported, the installer fails because of dependencies, that are not yet updated to support php 8.1.

Most packages can be easily updated with composer update however some dependencies need some extra steps and thought.

The concerned dependencies are:

The mezzio/mezzio-aurarouter probably won't be able to support php 8.1 in the near future as the aura project seems to be abandoned. The aura projects don't seem to be actively maintained for quite some time now. Also their mailing list seems to be pretty dead (last mail on January 28th).

Current behavior

composer create-project mezzio/mezzio-skeleton fails, as composer cannot resolve an installable set of packages. Afterwards composer update also fails, because of the aforementioned libraries.

How to reproduce

php version: 8.1 composer version: 2.1.5

Run composer create-project mezzio/mezzio-skeleton, after it fails run composer update inside the project.

Expected behavior

composer create-project mezzio/mezzio-skeleton runs successfully and creates the expected project.

Additional thoughts

I have already fixed the issues and will create PR in the next minutes, which I will link in the next comment.

While working on it, I also took #58 and #53, as for one I had to update mezzio/mezzio-tooling and second I had to fix a lot of tests, which where also failing for the auryn container. As there already exists the issue for it, I just removed it on the way.

Ocramius commented 2 years ago

@lukaslangen if we upgade mezzio/mezzio-tooling here, will it work, when not using symfony/dependency-injectio or the aura/ router?

lukaslangen commented 2 years ago

The PR is in #60

lukaslangen commented 2 years ago

@Ocramius Can you elaborate a bit further? I don't quite understand your question right now

Ocramius commented 2 years ago

@lukaslangen I think my question is redundant: patch provides all answers :)

weierophinney commented 2 years ago

Completed in #60