search

bovigo / vfsStream

vfsStream is a stream wrapper for a virtual file system that may be helpful in unit tests to mock the real file system. It can be used with any unit test framework, like PHPUnit or SimpleTest.
BSD 3-Clause "New" or "Revised" License
1.42k stars 102 forks source link

PHP 8.2 Support - track issues here #271

Open allejo opened 2 years ago

allejo commented 2 years ago

Welp, here we go again! The v1.6.x-dev version available via Composer now has support for PHP 8.2 nightly (which is currently in development). We have added PHP 8.2 to our CI testing on the v1.6 branch and will maybe keep things functional while 8.2 is in development. Once PHP 8.2 hits RC status, then we'll work on fixing any remaining issues before the final release.

If you're developing for PHP 8.2 and run into issues with vfsStream, report them here!

As always, thank you for streaming with vfsStream 📂

Berdir commented 2 years ago

We're getting these deprecations in Drupal:

13x: Creation of dynamic property org\bovigo\vfs\vfsStreamWrapper::$context is deprecated

I believe this must be a public property on a stream wrapper class that is directly set by php. This is also defined in stream wrappers in Drupal and guzzle for example.

Berdir commented 2 years ago

Ah, this has been fixed by @neclimdul in https://github.com/bovigo/vfsStream/commit/17b0f39b4a767bf950066dbad1a9a0b0ec4f7c68. A new release with that fix would be great.

allejo commented 2 years ago

A new release with that fix would be great.

Woops, looks like I didn't make a GH release but the tag exists, v1.6.11-alpha.0; I pushed up the tag back in June.

Edit: I have since created the GH release for visibility

lolli42 commented 2 years ago

Can we have a final v1.6.11 release or should we switch to the alpha tag for now?

andypost commented 2 years ago

Please create a stable release, API for PHP 8.2 is frozen and beta1 is out, all tests passing

allejo commented 2 years ago

Officially tagged v1.6.11 🎉

andypost commented 2 years ago

Thank you a lot! it unblocks us