commit | 3b0abf2fb146af025440a48e1f7423595b1a5bfb | [log] [tgz] |
---|---|---|
author | Quinten Yearsley <qyearsley@chromium.org> | Wed Apr 14 22:48:50 2021 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Apr 14 22:48:50 2021 |
tree | 51abf2a3ebd9d926a5095592f2e7718b815648b5 | |
parent | 21bf78c3cd7800e121a1abf13e2c117a67e55ba0 [diff] |
tricium: Simplify and cleanup after supporting only recipe-based analyzers Several proto fields are now unused since all projects that use Tricium have now switched to only recipe-based analyzers: - analyzer "config defs", which were used to pass extra flags to legacy analyzers - swarming dimensions in workflow - isolator functions - analyzer cipd packages Also, since we know that we don't plan to add features to Tricium before merging with CV, we can just remove some things that were never used, e.g. "deadline" and worker "service_account". This also updates comments in proto files, and adds the recommended "go_package" to the top of proto files. Note that presubmit now gives go vet warnings that we want to ignore but I couldn't find a mechanism to ignore these, so presubmit is skipped. See crrev.com/c/2809677. No-Presubmit: true Bug: 1146109 Change-Id: I8b5b59bd8c2534ee2510f5bbb6394e7abfe70616 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2826838 Commit-Queue: Quinten Yearsley <qyearsley@chromium.org> Reviewed-by: Andrii Shyshkalov <tandrii@google.com> Cr-Commit-Position: refs/heads/master@{#40171}
Testing Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.from infra.path_hacks.common import <stg>
is actually getting <stg>
from build/scripts/common.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, run your tests with test.py:
*_test.py
files to this directory.Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have __init__.py
files in each directory of your module so Python recognizes it as a package.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style, except functions use lowercase_with_underscores
. Use yapf (git cl format
) to autoformat new code.