Skip to content

Conversation

@milmazz
Copy link
Member

@milmazz milmazz commented May 27, 2015

Reference: #215

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use File.rm_rf!(output). If it fails we will get a better reason than badmatch. :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your feedback @josevalim

Raise a File.Error in case of failure. Refs: elixir-lang#215
josevalim added a commit that referenced this pull request May 27, 2015
Delete output directory before generate docs.
@josevalim josevalim merged commit bbf4114 into elixir-lang:master May 27, 2015
@josevalim
Copy link
Member

❤️ 💚 💙 💛 💜

@milmazz milmazz deleted the rm_output_dir branch June 3, 2015 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants