Hello we re integrating black with pants in a django project

Question

Hello we re integrating black with pants in a django project and we want all the migrations file to be skipped from black we already have in our `pyproject toml` the following ```exclude = | git | mypy cache | venv | migrations | protos ``` but running ` pants lint ` it does not skip them while running `black ` it does We also tried writing the following in the root BUILD file ```python sources name= migrations skip black=True sources= migrations py ``` but it didn t work We know we can put the `skip black=True` in each of the `BUILD` files inside all `migrations` folders but this would require us to add it every time we have a new `migrations` folder which is a bit of a pain Can you help us

bitter-ability-32190 · Answer

The issue I think is Black doesn t respect `exclude` if you provide filenames to the cmd I think they added a `force exclude` option for this

sparse-lifeguard-95737 · Answer

if you re using Pants gt = 2 14 you could use ` defaults ` to set `skip black=True` for all of your migrations

sparse-lifeguard-95737 · Answer

something like ``` In migrations BUILD pants defaults python sources python source dict skip black=True ```

quick-iron-62162 · Answer

< sparse lifeguard 95737> The fact is we have a lot of folders containing a `migrations` sub folder

sparse-lifeguard-95737 · Answer

ah hrm

quick-iron-62162 · Answer

< bitter ability 32190> thanks for the hint we ll try

quick-iron-62162 · Answer

< bitter ability 32190> it worked tada thanks a lot

bitter-ability-32190 · Answer

FWIW you ll be doing some wasted work here invoking Pants to run black and black then no ops but man shrugging

quick-iron-62162 · Answer

true that do you know if there is a way to write some defaults in the root build file to skip them

bitter-ability-32190 · Answer

Hmmmmmmmmm

bitter-ability-32190 · Answer

No and I m thinking of th elowest bar way of doing it and I got nothing easy

quick-iron-62162 · Answer

ok thanks

curved-television-6568 · Answer

I think you could get away with excludes and pants aliases ``` pants toml cli alias exclude migrations = src apps foo migrations src apps bar migrations ``` Then when formatting ``` pants fmt exclude migrations src apps ```

happy-kitchen-89482 · Answer

What you had could work ```python sources name= migrations skip black=True sources= migrations py ``` but I m guessing that you also have another `python sources` that also globs over the migrations

quick-iron-62162 · Answer

We also have some BUILD files created by ` pants tailor` in all of the `migrations` folders and all of them contains a `python sources ` so I guess they all override it am I correct

quick-iron-62162 · Answer

< happy kitchen 89482> We just needed to remove all ` migrations BUILD` or am I taking a wrong conclusion

happy-kitchen-89482 · Answer

That would work yes And since that higher level `migrations` target would cover new migration dirs as well tailor wouldn t generate new BUILD files for those new migrations

quick-iron-62162 · Answer

Great thanks a lot