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.

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

Drop Flysystem 1 and PHP 7.4 #185

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Drop Flysystem 1 and PHP 7.4 #185

wants to merge 3 commits into from

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Jul 5, 2022

No description provided.

@Nyholm
Copy link
Member Author

Nyholm commented Jul 5, 2022

TODO, I think we should remove our filesystem interface and make Flysystem a first class citizen.

@Nyholm Nyholm mentioned this pull request Jul 9, 2022
Closed
@xkzl
Copy link

xkzl commented Jul 10, 2022

Speaking about stale features, if you think to rebuild something centered around flysystem bundle.

Perhaps it would be good to consider doing the same with doctrine.
I mean that if you configure directly based on flysystem.yaml configuration, database might be configured the same way.

I guess the relevant part of backup-manager would be the shell processing class and compressor part.
Going one step further, compressor might also be replaced by Zippy or some other.
What do you think ?

@Nyholm Nyholm mentioned this pull request Aug 11, 2022
Open
@hyptos
Copy link

hyptos commented Mar 15, 2023

Hello,

Could we have an updates on this PR ?

Thanks

@Jelle-S
Copy link

Jelle-S commented Apr 3, 2023

@Nyholm This is blocking some stuff for us, any idea when we'll be able to use this, or what needs to be done to get this moving forward? Happy to help where I can.

Jelle-S added a commit to district09/backup-manager that referenced this pull request Apr 4, 2023
@Jelle-S
Forking form backup-manager/backup-manager
48fd345 
Continuing the work started at
backup-manager#185
This was referenced Apr 4, 2023
Closed
Merged
@xkzl
Copy link

xkzl commented Apr 4, 2023

In case someone is interested in, I am using a fork I have been developing in the last month:
https://gitlab.glitchr.dev/public-repository/agnostic/backup-manager

Jelle-S added a commit to district09/backup-manager that referenced this pull request Apr 4, 2023
@Jelle-S
Forking form backup-manager/backup-manager (#1)
47f82da 
* Forking form backup-manager/backup-manager

Continuing the work started at
backup-manager#185

* Fix ci issues

* Update to new namespace
Jelle-S added a commit to district09/backup-manager that referenced this pull request Apr 13, 2023
@Jelle-S
Merge branch 'develop'
29efc93 
Merge details:

commit 47f82da
Author: Jelle Sebreghts <sebreghts.jelle@gmail.com>
Date:   Tue Apr 4 16:07:00 2023

    Forking form backup-manager/backup-manager (#1)

    * Forking form backup-manager/backup-manager

    Continuing the work started at
    backup-manager#185

    * Fix ci issues

    * Update to new namespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Nyholm @xkzl @hyptos @Jelle-S