Patchwork D7202: fsmonitor: remove pywatchman from exclusion rule

login
register
mail settings
Submitter phabricator
Date Nov. 2, 2019, 10:04 p.m.
Message ID <differential-rev-PHID-DREV-froonz7memsmwtw7oper-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/42688/
State Superseded
Headers show

Comments

phabricator - Nov. 2, 2019, 10:04 p.m.
indygreg created this revision.
Herald added subscribers: mercurial-devel, mjpieters.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  The recently vendored pywatchman code base is now formatted
  with black. We can now remove pywatchman from our black
  exclusion rule.

REPOSITORY
  rHG Mercurial

BRANCH
  stable

REVISION DETAIL
  https://phab.mercurial-scm.org/D7202

AFFECTED FILES
  black.toml
  contrib/examples/fix.hgrc
  tests/test-check-format.t

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mjpieters, mercurial-devel

Patch

diff --git a/tests/test-check-format.t b/tests/test-check-format.t
--- a/tests/test-check-format.t
+++ b/tests/test-check-format.t
@@ -1,5 +1,5 @@ 
 #require black
 
   $ cd $RUNTESTDIR/..
-  $ black --config=black.toml --check --diff `hg files 'set:**.py - hgext/fsmonitor/pywatchman/** - mercurial/thirdparty/** - "contrib/python-zstandard/**"'`
+  $ black --config=black.toml --check --diff `hg files 'set:**.py - mercurial/thirdparty/** - "contrib/python-zstandard/**"'`
 
diff --git a/contrib/examples/fix.hgrc b/contrib/examples/fix.hgrc
--- a/contrib/examples/fix.hgrc
+++ b/contrib/examples/fix.hgrc
@@ -6,4 +6,4 @@ 
 rustfmt:pattern = set:**.rs
 
 black:command = black --config=black.toml -
-black:pattern = set:**.py - hgext/fsmonitor/pywatchman/** - mercurial/thirdparty/** - "contrib/python-zstandard/**"
+black:pattern = set:**.py - mercurial/thirdparty/** - "contrib/python-zstandard/**"
diff --git a/black.toml b/black.toml
--- a/black.toml
+++ b/black.toml
@@ -9,7 +9,6 @@ 
 | \.mypy_cache/
 | \.venv/
 | mercurial/thirdparty/
-| hgext/fsmonitor/pywatchman/
 | contrib/python-zstandard/
 '''
 skip-string-normalization = true