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

Encourage the use of "league/flysystem-sftp-v3" instead of "league/fl… #97

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

maxhelias
Copy link
Collaborator

…ysystem-sftp"

@maxhelias
Copy link
Collaborator Author

related to thephpleague/flysystem#1261

@maxhelias
Copy link
Collaborator Author

friendly ping @tgalopin 😃

@tgalopin
Copy link
Member

The PHP 7.2 / Symfony 4.4 tests are failing, I guess because the version installed if probably old. Maybe it's time to release a new major dropping support for EOL tools. WDYT?

@maxhelias
Copy link
Collaborator Author

Oh I didn't pay attention to the CI in 7.2. It's related to the version of flysystem which had a bad configuration of the .gitatttributes in 2.0.0 and still includes phpseclib v2

@maxhelias
Copy link
Collaborator Author

Not necessarily need to drop support now but I'm always for that kind of thing, I can work on it on another PR

@tgalopin
Copy link
Member

Thanks for the fix :) ! Merging

@tgalopin tgalopin merged commit 77a3dfb into thephpleague:master Jun 17, 2022
@tgalopin
Copy link
Member

tgalopin commented Jun 17, 2022

If you wish to open a PR for Symfony 6+ and PHP 8+, that would be great :) !

@maxhelias maxhelias deleted the sftp-v3 branch July 25, 2022 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants