bitwalker / distillery

Simplify deployments in Elixir with OTP releases!
MIT License
2.96k stars 396 forks source link

Confusing/incorrect docs around --warnings-as-errors flag #655

Closed bglusman closed 5 years ago

bglusman commented 5 years ago

Based on comments in docs code at https://github.com/bitwalker/distillery/blob/master/lib/distillery/tasks/release.ex I would expect mix release --warnings-as-errors to fail when the release contains warnings, but this was not the case in a test with an intentionally introduced warning.

Discussion on Slack suggested simply using mix compile --force --warnings-as-errors && mix release which works fine, but if this flag isn't supported by distillery for release task, ideally docs should be updated, unless I'm misunderstanding them of course.

bitwalker commented 5 years ago

Ah, yes --warnings-as-errors is a different flag for the release task which treats release warnings as errors, not compilation warnings as errors. For that, the command you've used as a workaround is the correct solution: specifically, run compile first, then run release.

I will update the docs to make note of this.