commit 35d9e716a76652f4c206fc9e968a07f67d7bdf1d
author Rob Mohr <mohrr@google.com> Wed May 26 13:41:45 2021 -0700
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 27 16:21:54 2021 +0000
tree 85eaebd30aec755c4a50a8cfc86ab9091747d589
parent 405b1468a5b0550c27a35733f561828b1c58a38d

pw_presubmit: Ignore yarn.lock in inclusive check

Bug: 386
Change-Id: I28dc296df331ceb1a35fd6ed47352dd3bcc78bc2
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/47140
Pigweed-Auto-Submit: Rob Mohr <mohrr@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Reviewed-by: Wyatt Hepler <hepler@google.com>

pw_presubmit/py/pw_presubmit/pigweed_presubmit.py

diff --git a/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py b/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
index 7c25990..5ec9622 100755
--- a/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
+++ b/pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
@@ -36,8 +36,8 @@
 import pw_package.pigweed_packages
 
 from pw_presubmit import build, cli, format_code, git_repo, call, filter_paths
-from pw_presubmit import inclusive_language, plural, PresubmitContext
-from pw_presubmit import PresubmitFailure, Programs
+import pw_presubmit.inclusive_language
+from pw_presubmit import plural, PresubmitContext, PresubmitFailure, Programs
 from pw_presubmit.install_hook import install_hook
 
 _LOG = logging.getLogger(__name__)
@@ -611,6 +611,11 @@
         raise PresubmitFailure
 
 
+@filter_paths(exclude=(r'\byarn.lock$', ))
+def inclusive_language(ctx: PresubmitContext):
+    pw_presubmit.inclusive_language.inclusive_language(ctx)
+
+
 def renode_check(ctx: PresubmitContext):
     """Placeholder for future check."""
     _LOG.info('%s %s', ctx.root, ctx.output_dir)
@@ -621,7 +626,7 @@
 #
 
 OTHER_CHECKS = (
-    inclusive_language.inclusive_language,
+    inclusive_language,
     # TODO(pwbug/45): Remove clang-tidy from OTHER_CHECKS when it passes.
     clang_tidy,
     # Build that attempts to duplicate the build OSS-Fuzz does. Currently