commit | d19f1551b5ace99b6e3497bf2dfba663a87ef7de | [log] [tgz] |
---|---|---|
author | Takuto Ikuta <tikuta@chromium.org> | Tue Mar 23 16:49:37 2021 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Mar 23 16:49:37 2021 |
tree | 775b72381eaf86aa77e82146bd219b8d32e3114c | |
parent | 6f040e7390171655e27e1443d79d9c9e7cb24d69 [diff] |
go: remove unused deps I listed up unused deps with ``` for p in $(fgrep -- '- package' deps.yaml | cut -d' ' -f3) do rg -q --no-ignore-dot --no-ignore-vcs "\"$p" src/ .vendor/ || echo $p done ``` -------------------------------------------------------------------------- Dependencies removed from deps.lock: * github.com/asaskevich/govalidator * github.com/disintegration/imaging * github.com/fedesog/webdriver * github.com/golang/lint/golint * github.com/juju/errors * github.com/pelletier/go-buffruneio * github.com/stoewer/go-strcase * gopkg.in/natefinch/npipe.v2 -------------------------------------------------------------------------- -------------------------------------------------------------------------- Updated repos: https://code.googlesource.com/gocloud.git/+log/4a0a22e44ac8..ab4824a79148 https://chromium.googlesource.com/external/github.com/chromedp/cdproto.git/+log/733a37e2c059..0942afbea50e https://chromium.googlesource.com/external/github.com/chromedp/chromedp.git/+log/4dea8b9dd8ab..ac448d20810e https://chromium.googlesource.com/external/github.com/google/pprof.git/+log/cbba55b83ad5..b188cf6ae17b https://chromium.googlesource.com/external/github.com/klauspost/compress.git/+log/a0dc84a8cf24..1454288976f8 https://chromium.googlesource.com/external/github.com/pkg/sftp.git/+log/96bf4d30dc70..ec19f0847c52 https://go.googlesource.com/crypto.git/+log/513c2a44f670..0c34fe9e7dc2 https://go.googlesource.com/tools.git/+log/d7a25adab0d9..d8aeb16bb5b3 https://chromium.googlesource.com/external/github.com/googleapis/go-genproto.git/+log/93e7006c17a6..5f0e89347f5a https://chromium.googlesource.com/external/github.com/grpc/grpc-go.git/+log/bce1cded4b05..d26af8e39165 -------------------------------------------------------------------------- Change-Id: Icf5eb5d3f4f3a0bdff357d30ab568fea0f5bed53 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2781620 Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Vadim Shtayura <vadimsh@chromium.org> Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Cr-Commit-Position: refs/heads/master@{#39641}
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.