Hopefully a quick quality of life question but can anyone ex Pants #general

Hopefully a quick "quality of life" question but c...

prehistoric-afternoon-67621

03/25/2022, 4:27 PM

Hopefully a quick "quality of life" question but can anyone explain to me how to get

./pants fmt ::

with

black

to run on my BUILD files? I've seen reference to that ability but not sure how to wire it up. 😇

✅ 1

curved-television-6568

03/25/2022, 4:30 PM

Hi! It’s

./pants update-build-files

that does that for you.

🙌 1

👍 1

rhythmic-battery-45198

03/25/2022, 4:30 PM

AFAIK, formatting the BUILD files is part of the

update-build-files

goal

🎯 1

👍 1

rhythmic-battery-45198

03/25/2022, 4:31 PM

Any way to format

pants.toml

curved-television-6568

03/25/2022, 4:32 PM

No, that would require a 3rdparty tool..

✅ 1

rhythmic-battery-45198

03/25/2022, 4:32 PM

prehistoric-afternoon-67621

03/25/2022, 4:36 PM

Awesome thanks!!

👍 1

prehistoric-afternoon-67621

03/25/2022, 4:38 PM

I'm (just) now understanding why that command uses a different interpreter constraints than the

[python.interpreter_constraints]

option. 👍

hundreds-father-404

03/25/2022, 5:25 PM

ah yeah you can force it to use certain interpreter constraints with

[black].interpreter_constraints

hundreds-father-404

03/25/2022, 5:25 PM

btw there's been much discussion to merge

tailor

with

update-build-files

, and then possiblyyyy to merge some of that with

fmt

prehistoric-afternoon-67621

03/25/2022, 5:26 PM

Fwiw I don't mind formatting the BUILD files with a separate command. But thanks for the context!

❤️ 1

prehistoric-afternoon-67621

03/25/2022, 5:27 PM

My situation was that I had tool lockfile for black that reflected my

[python].interpreter_constraints

but when I would try to run

./pants update-build-files

it would fail because the lockfile didn't match

[black].interpreter_constraints

. Minor edge case but was confusing to me until now

👍 1

👀 1

hundreds-father-404

03/25/2022, 5:28 PM

Would you mind opening a ticket explaining that edge case please? That's definitely confusing. https://github.com/pantsbuild/pants/issues/new/choose

polite-garden-50641

03/25/2022, 5:28 PM

I did:

find . -name BUILD | xargs black

locally and

find . -name "BUILD" | xargs python -m black --check

in ci (you have to have black installed locally or in CI somehow). I didn't try to move the new update build files goal....

prehistoric-afternoon-67621

03/25/2022, 5:29 PM

@hundreds-father-404 will do!

🙌 1

prehistoric-afternoon-67621

03/25/2022, 6:34 PM

Alright, I did my best to capture what's going on here. https://github.com/pantsbuild/pants/issues/14912

❤️ 1

3 Views

Open in Slack

Previous Next