Jelle Zijlstra

Wow, that's pretty unfortunate. It's meant to *split* strings that are too long, not to make new oversized strings. We should track this as a bug and fix it before...

The SQLAlchemy bug alluded to above is #2271.

I agree with adding parentheses in this context. Also, yes it would be good to start a new issue.

This is still broken (noting this because a similar bug with the magic trailing comma was fixed).

#2498 has some examples showing this is more general: if a statement contains multiple magic trailing commas, the magic only extends to one of them.

This has some conflicts now. With our new stability policy in place, this would be a good candidate to go into the "unstable" flag for a year to see it...

This should be fixed in the latest version. What version of Click are you using?

We could also consider blaming `click`, because `click.echo` is supposed to handle Unicode correctly for us: https://click.palletsprojects.com/en/7.x/api/#click.echo

The number of changes seem surprisingly small. Only a few hundred out of 10k files have a multi-line docstring?

This also happens with `# yapf: disable` (#2944).