Skip to content

Conversation

@WyriHaximus
Copy link
Member

Fixes / Closes #38

@WyriHaximus WyriHaximus force-pushed the binairy-file-read-write branch from 39b9300 to 9ce450a Compare August 1, 2018 18:35
@WyriHaximus
Copy link
Member Author

Ping @jsor @clue, rebased it on master and tests are passing

@clue
Copy link
Member

clue commented Aug 2, 2018

Not a big fan of the overhead that base64 encoding incurs, but seeing it fixes a relevant bug, I'm okay with this for now 👍

@jsor jsor merged commit 7ae9728 into reactphp:master Aug 2, 2018
@WyriHaximus
Copy link
Member Author

@clue neither am I, but I'm up for suggestions in 0.2.0. One could be two methods, one for text files and one for binary files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants