Skip to content
This repository has been archived by the owner on May 5, 2024. It is now read-only.

Implemented FlySystem for FileSystem abstraction #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dubcanada
Copy link

Solves #41

I wasn't exactly sure if this is the best method or not, but it works. Basically I took out the FileStorage interface and replaced it with FlySystem.

@scottrobertson
Copy link
Contributor

Thanks for this, shall take a look at it today.

@scottrobertson
Copy link
Contributor

Sorry, been very busy. @codeguy can you take a look at this too?

@scottrobertson
Copy link
Contributor

I am happy with this as a first step i think.

@dubcanada
Copy link
Author

Thoughts?

@codeguy
Copy link
Contributor

codeguy commented Sep 3, 2014

I like. @scottrobertson what else do you have in mind?

@scottrobertson
Copy link
Contributor

We should pick this up again, sorry i did not respond. Is this still ok to merge?

@bradleyboy
Copy link

Coming to this late but one thing to watch out for here is that FlySystem requires PHP 5.4+, while this library is currently baselined at 5.3.

@codeguy
Copy link
Contributor

codeguy commented Oct 15, 2014

@bradleyboy Good point. I'm fine bumping the requirements. I'll do that shortly.

@doorhinge
Copy link

@codeguy any update on this?

@doorhinge
Copy link

@codeguy Not to be annoying, but I'm really interested in using this with Flysystem, so I'm just wondering if you plan on merging this? Thanks 😄

@mikealmond
Copy link

Any update on this PR?

@tamtamchik
Copy link

👍 any ETA when this could possibly be implemented?

@jeremykendall
Copy link

Bump.

@kaperys
Copy link

kaperys commented Jan 15, 2016

Any update on this?

@napolux
Copy link

napolux commented Mar 30, 2016

Hellooooooo.... Is this repo still active?

@solleer
Copy link

solleer commented Jul 1, 2016

Will this be merged

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.