This is an archive of the discontinued Mercurial Phabricator instance.

formatting: format with newer black version
AbandonedPublic

Authored by martinvonz on Oct 14 2021, 4:11 PM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Summary

My black was upgraded from 20.8b1-4 to 21.4b2-3 today, which made
test-check-format.t fail. I don't know if we have specified a
particular version we're supposed to use. I also haven't tried to
downgrade black to see if the old version will complain about the new
format.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

martinvonz created this revision.Oct 14 2021, 4:11 PM
pulkit accepted this revision.Oct 15 2021, 8:41 AM
This revision is now accepted and ready to land.Oct 15 2021, 8:41 AM
pulkit added inline comments.Oct 15 2021, 9:08 AM
mercurial/localrepo.py
363

Hmm, I see it is reported as Invalid Syntax on Python 2.

martinvonz abandoned this revision.Oct 15 2021, 10:38 AM
martinvonz added inline comments.
mercurial/localrepo.py
363

Hmm, I see it is reported as Invalid Syntax on Python 2.

I guess they made the not-so-radical decision to drop support for Python 2. I'll drop this patch then I'll just add the failure to my mental list of test failures to ignore. It sounds like we're pretty close to dropping support for Python 2 now anyway.