diff options
author | Jordan Bayles <jophba@chromium.org> | 2021-06-18 11:38:39 -0700 |
---|---|---|
committer | Jordan Bayles <jophba@chromium.org> | 2021-06-18 18:50:14 +0000 |
commit | 5139930c36d7f9e6949bbd5d9ec254aa111683d8 (patch) | |
tree | e9e61a41ce49259354c5a46fb1065cc8eaaa76b4 | |
parent | 423f5afa0824473d502cc0d66f007ea000890df8 (diff) | |
download | openscreen-5139930c36d7f9e6949bbd5d9ec254aa111683d8.tar.gz |
Fix annoying PRESUBMIT errors
This patch silences the annoying PRESUBMIT error complaining about Open
Screen lacking a "main config set" and adds a reference to the bug to
actually fix the issue.
Change-Id: I46aee97619b3dd622af4c454e842b307b932b504
Reviewed-on: https://chromium-review.googlesource.com/c/openscreen/+/2973374
Commit-Queue: Jordan Bayles <jophba@chromium.org>
Reviewed-by: Ryan Keane <rwkeane@google.com>
-rwxr-xr-x | PRESUBMIT.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index a07514dc..66f91fd7 100755 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -211,8 +211,9 @@ def CheckChangeOnUpload(input_api, output_api): input_api.DEFAULT_FILES_TO_SKIP = _EXCLUDED_PATHS # We always run the OnCommit checks, as well as some additional checks. results = CheckChangeOnCommit(input_api, output_api) - results.extend( - input_api.canned_checks.CheckChangedLUCIConfigs(input_api, output_api)) + # TODO(crbug.com/1220846): Open Screen needs a `main` config_set. + #results.extend( + # input_api.canned_checks.CheckChangedLUCIConfigs(input_api, output_api)) return results |