Add support for .formatter.exs in subdirectories

[wingyplus]

When try to format Ecto migration script that generate project from
Phoenix, it will returns inputs "*.exs". But the glob list just detect
it only the root of project and apps directory which for umbrella
project. This cause make migration script won't match any glob pattern.

This changes fixes by add directory of file that want to format to the end
of glob list.

Fixes #361

[lukaszsamson]

Shouldn't it be added only when Path.dirname(file) differs from project_dir?

[wingyplus]

You're right. It should check file are in the project dir before add to the globs.

[wingyplus]

Fixed

[lukaszsamson]

Please, see comments below

[lukaszsamson]

Unfortunately it's still broken.

1. When formatting a file from top level project dir e.g. mix.exs, we end up expanding the same dirs twice

[
 "/Users/bug_repro/phoenix_ecto_formatter/*.{ex,exs}", # this
 "/Users/bug_repro/phoenix_ecto_formatter/apps/*/*.{ex,exs}",
 "/Users/bug_repro/phoenix_ecto_formatter/*.{ex,exs}" # and this
]

I meant that we should add something to globs only if it's not already there.
Probably we shouldn't also try to expand apps/* if it's not an umbrella project.
2. It doesn't work if the .formatter.exs is in a parent dir of the formatted file. In your repro project consider following:
.formatter.exs

[
  import_deps: [:ecto, :phoenix],
  inputs: ["*.{ex,exs}", "priv/*/seeds.exs", "{config,lib,test}/**/*.{ex,exs}"],
  subdirectories: ["priv/*/migrations", "priv/*"]
]

priv/repo/.formatter.exs (note it's moved one level up)

[
  import_deps: [:ecto_sql],
  inputs: ["migrations/*.exs"]
]

mix format works fine in that case
To fix it we probably need to check all parent dirs up to a project dir (or app dir in case of an umbrella) for relevant .formatter.exs files

[wingyplus]

@lukaszsamson thanks for your guide. I think it would be great to do this. Let me try. 🙏

[lukaszsamson]

@wingyplus ping

[wingyplus]

@lukaszsamson Hi, Sorry for disappear for a long time. I already see the WIP PR (but it's read in details yet), it's seems that it solve this issue?

[lukaszsamson]

@wingyplus is it still an issue?

[lukaszsamson]

Closing as this was superseded by #609