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.

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

no echo please! #12

Merged
merged 1 commit into from
Feb 16, 2014
Merged

no echo please! #12

merged 1 commit into from
Feb 16, 2014

Conversation

laughedelic
Copy link
Owner

This is definitely not a good way of writing to a file

https://github.com/laughedelic/literator/blob/master/lib/src/main/scala/FileUtils.scala#L61

and it crashes on windows

@ghost ghost assigned laughedelic Jan 8, 2014
@laughedelic laughedelic mentioned this pull request Feb 16, 2014
6 tasks
@laughedelic
Copy link
Owner

Thank you for the feedback @eparejatobes! Sorry for fixing it so slowly. I took the solution from an SO answer.
I think it's a shame for the Scala standard library, if such SO questions exist. And shame on me that I forgot about windows existence ")

@laughedelic laughedelic merged commit 17c3d1d into release/v0.5.0 Feb 16, 2014
@laughedelic laughedelic deleted the bugfix/no-echo branch February 16, 2014 23:33
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.

1 participant