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.

Sign up for GitHub

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

Jump to bottom

Replace System.stop/1 with Mix.raise/1 #23

Open

iamvery wants to merge 1 commit into Artur-Sulej:master from iamvery:iamvery/exit-status

iamvery commented Jan 4, 2023

[closes #22]

This should ensure a non-zero exit status is returned when dangers are detected.

Example:

» mix excellent_migrations.check_safety; echo $status

09:20:19.518 [warning] Index not concurrently in priv/repo/migrations/20230103220859_testing-stuff.exs:5
==> app
** (Mix) Dangerous operations detected in migrations!
1

and

» mix excellent_migrations.check_safety; echo $status

09:22:41.930 [info] No dangerous operations detected in migrations.
0


          Replace System.stop/1 with Mix.raise/1

This should ensure a non-zero exit status is returned when dangers are
detected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment