Overall I love that BUILD files are now linted with Black but I do kind of wish that it somehow favoured newlines more than it does.

pex_binary(name="manage", entry_point="manage.py", dependencies=["src/mysite:mysite", "src/polls:polls"], restartable=True)

VS

pex_binary(
    name="manage", 
    entry_point="manage.py",
    dependencies=[
        "src/mysite:mysite", 
        "src/polls:polls"
    ],
    restartable=True
)

I much prefer the second one - it makes diffs so much more readable. Guessing this is configurable if I look for the option but maybe we want to override the defaults on Black for this use case out of the box?

you can force newlines by adding trailing commas! Try rerunning where you put

,

after

restartable

Oh nice - TIL. Thank you! 🙏🙏

You'd think Black would be configurable but it's mostly not

This looks like a killer tip for the docs. Maybe even have Tailor comment about it?

We haven't documented

update-build-files

at all yet. That's something I'll be adding as part of the 2.8 release blog work