bitwalker/distillery

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

bglusman opened this issue · 1 comments

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.

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.