Pre-commit hook does not check test pass/failure before allowing commit ...

[nelsonic]

I'm trying to debug why a test is not passing for #35
and when I attempt to make a commit on my branch (where tests are failing)
I am not prevented from doing so ...

pre_commit is in the mix.exs file:

github-backup/mix.exs

Line 50 in 9aec5c4

{:pre_commit, "~> 0.2.4", only: :dev},

But it's unclear what command is being run on the pre-commit hook ...

github-backup/config/prod.exs

Lines 34 to 35 in 5033458

	# Configures pre-commit commands
	config :pre_commit, commands: ["credo --strict", "test"], verbose: true

Why is the pre-commit defined in prod.exs ?

It would be desirable to have the pre-commit run on localhost i.e. dev.exs ...
@SimonLab / @Cleop thoughts...?

[SimonLab]

Yes the pre-commit should be on the dev.exs config file 👍
I think we missed some steps on how pre-commit is run.
I'll review the pre-commit documentation (https://github.com/dwyl/elixir-pre-commit) and update the configuration file with the commands to run