python3 is the only supported version of python in depot_tools.
Changes include:
* updating python version requirements
* removing third_party/six
Bug: 1475402
Change-Id: Ib86231413fcac494ceb27d1783e53b9d86533c26
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4845292
Reviewed-by: Josip Sokcevic <sokcevic@chromium.org>
Commit-Queue: Gavin Mak <gavinmak@google.com>
This file is used by a few dependent projects. Highlighting py3
compatiblity eases the migration process for those projects.
This CL additionally adds executation permissions to the file and
includes the script in the docs.
Bug: 1298903
Change-Id: I5838fedcef181ee5e3eb84a1fa16b649250460ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3479854
Auto-Submit: Alexander Schulze <alexschulze@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
This also allows digits in class decorators, to also match macros like
V8_EXPORT_PRIVATE.
Note: I uploaded a pull request for the (previous) upstream at
https://github.com/google/styleguide/pull/422, but then was told that
that repository is not maintained any more.
R=jochen@chromium.org
Bug: chromium:916550
Change-Id: I24cd1829b794b296be8e0c2829271c746751ee1c
Reviewed-on: https://chromium-review.googlesource.com/c/1383472
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Specifically, this CL was made by running codespell
(https://github.com/lucasdemarchi/codespell), manually filtering
for changes in non-third-party files that appear correct.
Change-Id: Ia16c1b29483d777744450d7bea45a178cf877a25
Reviewed-on: https://chromium-review.googlesource.com/420871
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>
Reason for revert:
need another build :)
Original issue's description:
> Whitespace change to trigger cipd package for git installer.
>
> TBR=phajdan@chromium.org, vadimsh@chromium.org
> BUG=635421
>
> Committed: 31c2115e6a
TBR=phajdan.jr@chromium.org,vadimsh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=635421
Review-Url: https://codereview.chromium.org/2252983002
I didn't change the content much but deleted some obviously wrong stuff. Renamed
the files to .md so they will be processed by gitiles.
This change is in no way "sufficient" but that's a start.
R=tandrii@chromium.org
Review-Url: https://codereview.chromium.org/2232303002
Anne Redulla
Gavin Mak
Alexander Schulze
Andreas Papacharalampous
Quinten Yearsley
Clemens Hammacher
Robert Iannucci
anatoly techtonik
Quinten Yearsley
benhenry
maruel