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.

Sign up for GitHub

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

Jump to bottom

Flysysystem V2 and V3 #688

Closed

dmitryuk wants to merge 1 commit into KnpLabs:master from dmitryuk:fly-system-v3

dmitryuk commented Nov 25, 2022 •

edited

Loading

Added class FlysystemV2V3 - can be used on both versions.

Much of time spent to fix analizers.
Psalm upgraded to 4.30.0 to use new plugin system
CI flow divided to test and analyse different versions of flysystem

Inspired on #650


          Flysysystem V2 and V3

caf74a4

dmitryuk mentioned this pull request

[Flystem] Add FlysystemV2 adapter for version 2 support of flysystem. #650

Open

PedroTroller force-pushed the master branch from 148fafd to 3cc396c Compare

November 29, 2022 09:30

Author

dmitryuk commented Dec 1, 2022

About Roave we can skip it, because originally Gaufrette changed names of arguments and now its it right.

[BC] CHANGED: Parameter 0 of Gaufrette\Adapter\InMemory#isDirectory() changed name from path to key
[BC] CHANGED: Parameter 0 of Gaufrette\Stream\InMemoryBuffer#cast() changed name from castAst to castAs

dmitryuk closed this

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

Labels

None yet