D6342: style: run a patched black on a subset of mercurial

2019-10-06 Thread durin42 (Augie Fackler)
Closed by commit rHG57875cf423c9: style: run a patched black on a subset of mercurial (authored by durin42). This revision was automatically updated to reflect the committed changes. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=16881&id=16902 CH

D6342: style: run a patched black on a subset of mercurial

2019-10-06 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 16881. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=16867&id=16881 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6342/new/ REVISION DETAIL https://phab.mercurial-scm.org/D6342 AFFECTED FILES

D6342: style: run a patched black on a subset of mercurial

2019-10-06 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 16867. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=16829&id=16867 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6342/new/ REVISION DETAIL https://phab.mercurial-scm.org/D6342 AFFECTED FILES

D6342: style: run a patched black on a subset of mercurial

2019-10-05 Thread indygreg (Gregory Szorc)
This revision is now accepted and ready to land. indygreg added a comment. indygreg accepted this revision. I'm going to split this on land so `pyproject.toml` appears in its own changeset. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6342/new/ REVI

D6342: style: run a patched black on a subset of mercurial

2019-10-05 Thread durin42 (Augie Fackler)
durin42 edited the summary of this revision. durin42 updated this revision to Diff 16829. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=16812&id=16829 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6342/new/ REVISION DETAIL https:/

D6342: style: run a patched black on a subset of mercurial

2019-10-05 Thread indygreg (Gregory Szorc)
This revision now requires changes to proceed. indygreg added a comment. indygreg requested changes to this revision. Please add `# skip-blame` to the commit message. And split the addition of `pyproject.toml` into its own commit so we have annotate info. REPOSITORY rHG Mercurial CHANGES S

D6342: style: run a patched black on a subset of mercurial

2019-10-05 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 16812. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=15040&id=16812 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6342/new/ REVISION DETAIL https://phab.mercurial-scm.org/D6342 AFFECTED FILES

D6342: style: run a patched black on a subset of mercurial

2019-05-08 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 15040. durin42 edited the summary of this revision. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=15039&id=15040 REVISION DETAIL https://phab.mercurial-scm.org/D6342 AFFECTED FILES contrib/import-checker.

D6342: style: run a patched black on a subset of mercurial

2019-05-08 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 15039. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6342?vs=15009&id=15039 REVISION DETAIL https://phab.mercurial-scm.org/D6342 AFFECTED FILES contrib/import-checker.py mercurial/cacheutil.py mercurial/diffh

Re: D6342: style: run a patched black on a subset of mercurial

2019-05-06 Thread Augie Fackler
Yes, the intent is to show what the result of the patch to black is, and once that lands to try and get consensus that we've addressed the major issue with black and can move on. I've tried adding b prefixes everywhere without black, or by using a different formatter, and the other formatters just

D6342: style: run a patched black on a subset of mercurial

2019-05-06 Thread martinvonz (Martin von Zweigbergk)
martinvonz added a comment. I would be happy to have our code auto-formatted. I assume this patch is just to show what the result will be and it's not meant to be queued until that patch for black has been accepted. INLINE COMMENTS > policy.py:27-30 > +b"c": ( > +r"cext", > +

D6342: style: run a patched black on a subset of mercurial

2019-05-04 Thread durin42 (Augie Fackler)
durin42 created this revision. Herald added subscribers: mercurial-devel, mjpieters. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This applied black to the 20 smallest files in mercurial/: ls -S1 mercurial/*.py | tail -n20 | xargs black Note that a few files failed to form