Don't skip magic-trailing-comma [black]

[cdce8p]

Description

Follow up to discussion in #13412 (comment)

Remove black setting to skip-maagic-trailing-comma since in some instances it can result in better readable code. With the setting enabled, black will always choose the most compact function definition even if it isn't the best solution.
https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#the-magic-trailing-comma

Initial comment adding it: #12424 (comment)

Test Plan

I've reapplied the changes suggested in #13412 (review) to show the impact of the setting change.

[hauntsaninja]

Thanks, but I thought I was pretty clear in #13412 (comment)

I'll leave this open for a couple days before closing, in case you get luck with a kinder maintainer than I :-)

[github-actions]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

[JelleZijlstra]

I'd also prefer to keep the current configuration. I only begrudgingly accepted the trailing comma going into Black in the first place: psf/black#826 (comment).

[AlexWaygood]

I'm not a maintainer here, but FWIW, I also prefer the current configuration.

[cdce8p]

Ok. Seems like most prefer the current look. I'll go ahead and close this then.