search

reactphp / filesystem

Evented filesystem access.
MIT License
135 stars 40 forks source link

Remove call invoker related classes #65

Closed WyriHaximus closed 4 years ago

ghost commented 5 years ago

@WyriHaximus 🏓 What's the current state?

WyriHaximus commented 5 years ago

@CharlotteDunois It's a rabbithole I need to get back into, got some stuff locally but there are a lot of changes and EIO is being an issue.

ghost commented 4 years ago

@WyriHaximus status?

WyriHaximus commented 4 years ago

@CharlotteDunois nothing really changes, going for a stab at it again tonight

clue commented 4 years ago

The less code, the better :+1: @WyriHaximus Can you add the appropriate labels (BC break etc.)?

WyriHaximus commented 4 years ago

The less code, the better @WyriHaximus Can you add the appropriate labels (BC break etc.)?

@clue done :+1:

ghost commented 4 years ago

@jsor 🏓

jsor commented 4 years ago

@WyriHaximus Tests are failing ❌

WyriHaximus commented 4 years ago

@jsor thought about it merging anyway and fixing that in a follow up PR where it is clear where the issues resides. This also lets @CharlotteDunois continue with https://github.com/reactphp/filesystem/pull/69