5
95%
master: 96%

Ran 25 Aug 2020 08:14PM CUT

Files 8

Run time 0s

Badge

Embed ▾

Committed 25 Aug 2020 08:10PM CUT coverage: 95.164%. First build

Job # TEST_CMD="coverage run -m unittest"

Build Type

push

travis-ci

Committed by

ambv

Commit Message

Address pre-existing trailing commas when not in the rightmost bracket pair

This required some hackery.  Long story short, we need to reuse the ability to
omit rightmost bracket pairs (which glues them together and splits on something
else instead), for use with pre-existing trailing commas.

This form of user-controlled formatting is brittle so we have to be careful not
to cause a scenario where Black first formats code without trailing commas in
one way, and then looks at the same file with pre-existing trailing commas
(that it itself put on the previous run) and decides to format the code again.

One particular ugly edge case here is handling of optional parentheses.  In
particular, the long-standing `line_length=1` hack got in the way of
pre-existing trailing commas and had to be removed.  Instead, a more
intelligent but costly solution was put in place: a "second opinion" if the
formatting that omits optional parentheses ended up causing lines to be too
long.  Again, for efficiency purposes, Black reuses Leaf objects from blib2to3
and modifies them in place, which was invalid for having two separate
formattings.  Line cloning was used to mitigate this.

Fixes #1619

Run Details

4683 of 4921 relevant lines covered (95.16%)

0.95 hits per line

Source Files on job 2073.5 (TEST_CMD="coverage run -m unittest")

94.44
src/
96.61
tests/

psf / black / 2073 / 5
95%
master: 96%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2073.5 (TEST_CMD="coverage run -m unittest")

psf / black / 2073 / 5 95% master: 96%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2073.5 (TEST_CMD="coverage run -m unittest")

psf / black / 2073 / 5
95%
master: 96%

README BADGES
x