akoutmos / doctor

Ensure that your Elixir project documentation is healthy
MIT License
179 stars 15 forks source link

CLI raising should impact exit code #57

Open christhekeele opened 1 year ago

christhekeele commented 1 year ago

Expectation:

With the raise: true config set, when running mix doctor, when my Doctor validation fails, I'd expect the task to return a non-zero status code.

Observed:

Failing and non-failing runs always return a 0 exit code.

Implementation:

One neat thing I've seen such tools do is encode the number of failures in the returned status code, so bash CI scripts can externally permit/enforce a certain number of failures.