commit | f2c71274458177b488027fdd9b3d0cbd384540ef | [log] [tgz] |
---|---|---|
author | Scott Lee <ddoman@chromium.org> | Wed Jan 06 09:13:32 2021 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Jan 06 09:13:32 2021 |
tree | c7c6f40b47ff74b0426a6f54db445f0ba59d06e9 | |
parent | 8e3415be1444041241c0b74b3d374d04cd597087 [diff] |
Revert "[gae_ts_mon] Replace Modules API with environment variables for py3 compatibility" This reverts commit 36c7b6704d17dbd1c7ee6f157040813eb7bda761. Reason for revert: broke unit tests - crbug.com/1163440 Original change's description: > [gae_ts_mon] Replace Modules API with environment variables for py3 compatibility > > Bug: monorail:8841 > Change-Id: I1c63e38191ea1983bc075360fe8a3220926f22c9 > Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2611839 > Commit-Queue: Kevin Weng <kweng@google.com> > Reviewed-by: Dave Tu <dtu@chromium.org> > Reviewed-by: Scott Lee <ddoman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#37311} TBR=dtu@chromium.org,ddoman@chromium.org,kweng@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com Change-Id: Ie717380459724c5b9a4daa096e00f7f17c5e4c3b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: monorail:8841 Bug: 1163440 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2612904 Commit-Queue: Scott Lee <ddoman@chromium.org> Reviewed-by: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#37320}
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.