Skip to content

Updates for flysystem3 - #39

Open
ylebre wants to merge 12 commits into
mainfrom
feature/flysystem-3
Open

ylebre wants to merge 12 commits into
mainfrom
feature/flysystem-3

Conversation

@ylebre

@ylebre ylebre commented Sep 28, 2026

Copy link
Copy Markdown
Member

No description provided.

@Potherca

Copy link
Copy Markdown
Member

Tests fail because of "Your requirements could not be resolved to an installable set of packages." error.

To fix this the league/flysystem needs top be changed to ^3.0 in composer.json.

However, the mjrider/flysystem-factory will then error out (because it needs v1).

The flysystem-factory package was abandoned. As I was attached to the original project, I can create a fork and update it.

@ylebre Do you have a preference to where the fork should live? (Under my personal Potherca account or under Muze-nl?)

@ylebre

ylebre commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

Hm, lets take a peek why we are using flysystem-factory in the first place? would it be hard not to?

@Potherca

Potherca commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Uhm, on closer inspection, the code from that package doesn't even seem to be used anywhere? 🤨

Just yank it.

This branch has not been deployed

No deployments
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