mono-repo with multiple applications about elixir_git_hooks

[jgallinari]

Sorry for the delay of my response.
I could not get it to work until I re-installed the git_hooks manually with mix git_hooks.install, and suddenly it started to work as expected 😮 : .git/hooks/pre-commit got the right cd command.
Thanks for your help!
I think you can close the ticket.

[qgadrian]

Hello @jgallinari! Thanks for reporting the issue!

I believe this is the same issue as #113 correct? If that is the case, I opened a PR to validate adding a config to set up the project would work #120

In that case, if this library it's available on both of your projects it should work for both of them.

[jgallinari]

Hello,
Not sure to understand how it's supposed to work 😞
In repo/app1/config/config.exs, I put:

if Mix.env() != :prod do
  config :git_hooks,
    project_path: "./app1",
    auto_install: true,
    verbose: true,
    hooks: [
      pre_commit: [
        tasks: [
          {:cmd, "mix credo"},

And then, by installing, I got

repo/app1> mix git_hooks.install
↗ Installing git hooks...
↗ Writing git hook for `pre_commit` to `../.git/hooks/pre-commit`
↗ Backing up git hook file `../.git/hooks/pre-commit` to `../.git/hooks/pre-commit.pre_git_hooks_backup`
↗ Writing git hook for `pre_push` to `../.git/hooks/pre-push`
↗ Backing up git hook file `../.git/hooks/pre-push` to `../.git/hooks/pre-push.pre_git_hooks_backup`

and repo/.git/hooks/pre-commit now contains:

#!/bin/sh

[ "./app1" != "" ] && cd "./app1"

mix git_hooks.run pre_commit "$@"
[ $? -ne 0 ] && exit 1
exit 0

which works.
But what am I supposed to do with repo/app2 then?
I don't see how repo/app1 and repo/app2 can co-exist with a common config in repo/.git/hooks/pre-commit, can't they?

[qgadrian]

Hello, Not sure to understand how it's supposed to work 😞 In repo/app1/config/config.exs, I put:

if Mix.env() != :prod do
  config :git_hooks,
    project_path: "./app1",
    auto_install: true,
    verbose: true,
    hooks: [
      pre_commit: [
        tasks: [
          {:cmd, "mix credo"},

And then, by installing, I got

repo/app1> mix git_hooks.install
↗ Installing git hooks...
↗ Writing git hook for `pre_commit` to `../.git/hooks/pre-commit`
↗ Backing up git hook file `../.git/hooks/pre-commit` to `../.git/hooks/pre-commit.pre_git_hooks_backup`
↗ Writing git hook for `pre_push` to `../.git/hooks/pre-push`
↗ Backing up git hook file `../.git/hooks/pre-push` to `../.git/hooks/pre-push.pre_git_hooks_backup`

and repo/.git/hooks/pre-commit now contains:

#!/bin/sh

[ "./app1" != "" ] && cd "./app1"

mix git_hooks.run pre_commit "$@"
[ $? -ne 0 ] && exit 1
exit 0

which works. But what am I supposed to do with repo/app2 then? I don't see how repo/app1 and repo/app2 can co-exist with a common config in repo/.git/hooks/pre-commit, can't they?

If it's an umbrella application, if both applications use this library you can have this library configured at the root config file instead of on each application.

If it's not an umbrella application I can try to dig more to support your case ☺️. It would be very helpful to speed up things if you can provide an example project to reproduce the issue.

[qgadrian]

@jgallinari glad to hear. If you have any issues please free to ping me here or open a new ticket ☺️