| commit | e91e4c82946cf6e61cc43fe2e0bca2ff70772e61 | [log] [tgz] |
|---|---|---|
| author | Greg Guterman <guterman@google.com> | Thu Oct 08 04:51:27 2020 |
| committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu Oct 08 04:51:27 2020 |
| tree | 51df56a3a1f3f25b92ddd03836701d957f228af9 | |
| parent | 8842995523409800045afc20aca13247a20a2a3d [diff] |
Rework analyze DEPS logic to run before testing begins Previously, this logic ran at the end of a build. This ran into the problem of the patch being deapplied in the "without changes" phase and thus this logic couldn't see which files were changed. Now it's split up to get a revised list of test targets during the analyze phase, while still doing the correctness checking at the end. Bug: 923016 Change-Id: I120ff9d463c05d02515ed81ca83005ad9cb42ca3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2424888 Commit-Queue: Gregory Guterman <guterman@google.com> Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style. Functions use lowercase_with_underscores, with the exception of the special functions RunSteps and RunTests in recipes. Use yapf (git cl format --no-clang-format) to autoformat new code.