Ravi Mistry
fbff329787
Add hcm@ to public API owners
...
BUG=skia:
NOTRY=true
Change-Id: I4d54591db0d50ceca02a87868fd125309bd4531e
Reviewed-on: https://skia-review.googlesource.com/7275
Commit-Queue: Ravi Mistry <rmistry@google.com>
Reviewed-by: Mike Reed <reed@google.com>
2017-01-19 17:36:06 +00:00
Ravi Mistry
8c6a43a1c0
Do not use post upload hooks for 'GOLD_TRYBOT_URL'
...
That information will be provided via a plugin soon.
# This code is not exercised by trybots.
NOTRY=true
BUG=skia:6000
Change-Id: I6ea60783c0581e32317a781e4349e95b38b71da0
Reviewed-on: https://skia-review.googlesource.com/5207
Commit-Queue: Ravi Mistry <rmistry@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
2016-11-23 18:34:23 +00:00
Eric Boren
508313064b
[nobuildbot] Migrate all Linux GCE bots
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4994
Change-Id: Ib63f36b3803117d5357c8e70c572653afd151abe
Reviewed-on: https://skia-review.googlesource.com/4994
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
2016-11-23 14:49:14 +00:00
Mike Klein
a5fb6157e5
Split SkSL and utils apart again into .gni files.
...
Also extend GN format presubmit check to .gni files.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3976
Change-Id: I292f3136437101e3e2c3271e260536ab7a24b192
Reviewed-on: https://skia-review.googlesource.com/3976
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
2016-10-26 18:56:37 +00:00
borenet
2dbbfa5d37
Add infra_tests.py, recipe, buildbotless bot
...
Have the presubmit (upload) run infra_tests.py if anything in infra/ changed.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2415193002
Review-Url: https://codereview.chromium.org/2415193002
2016-10-14 06:32:09 -07:00
Mike Klein
7a1c53d0f6
Fix GN formatting presubmit on Windows.
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3182
Change-Id: I6e19b18d6851516357f71930e406b9fb324ea81f
Reviewed-on: https://skia-review.googlesource.com/3182
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Herb Derby <herb@google.com>
2016-10-11 18:13:17 +00:00
Ravi Mistry
dbb84c268b
Fix public API msg in PRESUBMIT.py
...
BUG=skia:5825
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2989
Change-Id: Ia074ab00ccd1500bee10a5b313636581e8265461
Reviewed-on: https://skia-review.googlesource.com/2989
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2016-10-05 17:08:55 +00:00
Ravi Mistry
39eabb6ccb
PRESUBMIT.py improvements
...
* Updated code to check for owners in reviewers for Gerrit issues instead for in the TBR= line.
* The previous TBR parsing code was not accounting for adding comments. Eg: "TBR=rmistry (Testing only)".
BUG=skia:5825
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2981
Change-Id: I910a3ae71a9f57c14f80c0b0404041cbe451a77c
Reviewed-on: https://skia-review.googlesource.com/2981
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2016-10-05 16:38:38 +00:00
borenet
60b0a2d85c
Add --test to gen_tasks.go, add presubmit check
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2392083002
Review-Url: https://codereview.chromium.org/2392083002
2016-10-04 12:45:41 -07:00
mtklein
9f262811b7
To prep for removing CMake bots, take them off the CQ and presubmit.
...
GN is now far more capable than our CMake builds.
Time to officially end that experiment.
Reminder how this lands:
1) this CL
2) remove the bots from the masters
3) remove the CMake recipe files
4) remove the CMake files
I'm willing to wait on 4) until we can do GN -> CMake generation, but I don't want the bots to hold us back.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350073002
Review-Url: https://codereview.chromium.org/2350073002
2016-09-19 09:08:07 -07:00
rmistry
07f0446e9a
Fix PostUploadHook for Gerrit issues
...
A gerrit change must contain the "Change-Id" at the end. This CL strips
out the changeId from the description and adds it back in at the end.
BUG=skia:5676
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2299303002
NOTRY=true
Review-Url: https://codereview.chromium.org/2299303002
2016-09-01 13:36:31 -07:00
rmistry
db0283bec9
Enable Gerrit support in Skia's PostUploadHook
...
BUG=skia:5676
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2299203002
NOTRY=true
Review-Url: https://codereview.chromium.org/2299203002
2016-09-01 09:54:01 -07:00
rmistry
b9a9e8788f
Use methods from depot_tool's Changelist object instead of Rietveld RPC
...
This will make support for both Gerrit and Rietveld easier.
BUG=skia:5676
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2307593002
NOTRY=true
Review-Url: https://codereview.chromium.org/2307593002
2016-09-01 09:52:32 -07:00
mtklein
04fff6373e
Fix up presubmit for SAN GCC->Clang fix.
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2302553002
Review-Url: https://codereview.chromium.org/2302553002
2016-08-31 12:43:14 -07:00
rmistry
b398ecc2a4
Add Gerrit API support to Skia's PRESUBMIT.py
...
Also skip post upload hooks for Gerrit.
BUG=skia:5674
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2281123002
Review-Url: https://codereview.chromium.org/2281123002
2016-08-29 08:13:29 -07:00
mtklein
d434b01c7e
Roll GN: gn format is --in-place by default
...
As usual, might as well roll, and this new behavior is a small convenience.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2231833002
Review-Url: https://codereview.chromium.org/2231833002
2016-08-10 07:30:58 -07:00
mtklein
4db3b7969f
Require gn format in presubmit
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2203283003
Review-Url: https://codereview.chromium.org/2203283003
2016-08-03 14:18:23 -07:00
mtklein
3da80f5ecd
Only run recipe simulation tests if infra/ changes.
...
This should lessen the need for mortals to have coverage.py,
and saves 1.8s (of 3.6s) on my desktop for this non-infra change.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2187713002
Review-Url: https://codereview.chromium.org/2187713002
2016-07-27 04:14:07 -07:00
borenet
1ed2ae45f5
Move Skia recipes from build repo
...
BUG=skia:5563
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2175373002
Review-Url: https://codereview.chromium.org/2175373002
2016-07-26 11:52:17 -07:00
rmistry
d8f0988b34
Specifying master names with "master." prefix is deprecated
...
NOTRY=true
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2151623002
Review-Url: https://codereview.chromium.org/2151623002
2016-07-13 12:57:42 -07:00
rmistry
3b3ee20d93
Do not use the deprecated CQ_EXTRA_TRYBOTS
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135063003
Review-Url: https://codereview.chromium.org/2135063003
2016-07-11 08:27:49 -07:00
rmistry
d88b0bec43
Fix Gold trybot results URL
...
BUG=skia:5324
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=1996103002
Review-Url: https://codereview.chromium.org/1996103002
2016-05-20 03:50:01 -07:00
mtklein
bb5f309bce
10.9 is no more
...
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1732353002
Review URL: https://codereview.chromium.org/1732353002
2016-02-25 06:52:18 -08:00
borenet
d9fa758292
Port Skia recipe to normal Python scripts, move to Skia repo
...
BUG=skia:4763
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1703663002
Review URL: https://codereview.chromium.org/1703663002
2016-02-18 08:05:48 -08:00
rmistry
11f457e162
Add link to Gold trybot runs in every uploaded Skia CL
...
BUG=skia:4433
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1521913005
Review URL: https://codereview.chromium.org/1521913005
2015-12-15 06:23:33 -08:00
halcanary
6950de6c41
Comments Style: s/skbug.com/bug.skia.org/
...
DOCS_PREVIEW= https://skia.org/?cl=1432503003
Review URL: https://codereview.chromium.org/1432503003
2015-11-07 05:29:00 -08:00
mtklein
ab8296b48a
Automatically add TSAN bots to the CQ for changes affecting SkAtomics.h
...
BUG=skia:
Review URL: https://codereview.chromium.org/1396523002
2015-10-07 11:49:00 -07:00
rmistry
5827653e42
Automatically add extra CQ trybots for predetermined paths
...
If the CL does not contain a "CQ_EXTRA_TRYBOTS=" section then it constructs one from the paths modified in your CL.
If you already have an existing "CQ_EXTRA_TRYBOTS=" section in your CL then it parses it, preserves your options, and only adds the builders and masters that were missing.
Also ensures no trybots are repeated (i.e. each master trybot combination only shows up once).
And I killed the hashtags feature in this CL, nobody was using it.
BUG=skia:4364
Review URL: https://codereview.chromium.org/1383743002
2015-10-01 08:24:03 -07:00
rmistry
9806d4ded5
Resolve TODOs in PRESUBMIT.py
...
BUG=skia:
Review URL: https://codereview.chromium.org/1384453004
2015-10-01 08:10:54 -07:00
mtklein
bda1267770
update presubmit to know about include/private
...
BUG=skia:4126
Review URL: https://codereview.chromium.org/1259953004
2015-07-28 08:54:12 -07:00
Mike Klein
478c9e4851
Revert "Move headers used by headers in include/ to include/private."
...
This reverts commit 928e16565f
.
BUG=skia:
Review URL: https://codereview.chromium.org/1213093004 .
2015-07-01 16:35:59 -04:00
mtklein
928e16565f
Move headers used by headers in include/ to include/private.
...
Some of this is transitive, like SkRecords.h used by SkMiniRecorder.h
used by (public) SkPictureRecorder.h.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/a89f55198bdc58f0b6f6196907ab25a6afc1a661
Review URL: https://codereview.chromium.org/1217293004
2015-07-01 11:55:42 -07:00
jvanverth
4417c7f8bb
Revert "Move headers used by headers in include/ to include/private."
...
This reverts commit a89f55198b
.
Reason: breaking the roll.
TBR=mtklein@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1216033008
2015-07-01 09:45:46 -07:00
mtklein
a89f55198b
Move headers used by headers in include/ to include/private.
...
Some of this is transitive, like SkRecords.h used by SkMiniRecorder.h
used by (public) SkPictureRecorder.h.
BUG=skia:
Review URL: https://codereview.chromium.org/1217293004
2015-07-01 08:41:15 -07:00
borenet
af6005c8a2
Presubmit: add PRESUBMIT=false to non-master branch changes
...
BUG=skia:
Review URL: https://codereview.chromium.org/1161973005
2015-06-04 15:15:42 -07:00
mtklein
e4b19c4593
Add presubmit to run {dm,nanobench}_flags.py test
when changed.
...
BUG=skia:
Review URL: https://codereview.chromium.org/1129443005
2015-05-05 10:28:44 -07:00
rmistry
04abf130bb
Bypass owners check for dry run CLs.
...
BUG=skia:
NOTRY=true
Review URL: https://codereview.chromium.org/1068853002
2015-04-07 07:41:51 -07:00
rmistry
57291dcc95
Find the HASHTAGS file even if you are not at the checkout root.
...
The post upload hook was failing if you were not at the root, this fixes it.
NOTRY=true
Review URL: https://codereview.chromium.org/1058483002
2015-04-01 09:12:51 -07:00
rmistry
3cfd1ad6c7
Add post upload hook to substitute hashtags for their mapped text
...
BUG=skia:3586
NOTRY=true
Review URL: https://codereview.chromium.org/1004733009
2015-03-25 12:53:36 -07:00
mtklein
18e55800c6
More file types to check for (C).
...
- objective C++ (31)
- objective C (4)
- C (2)
- alternate C++ extension (1)
- Go (1)
BUG=skia:
Review URL: https://codereview.chromium.org/1032143002
2015-03-25 07:21:20 -07:00
borenet
c7c91806e3
Add copyright check to PRESUBMIT
...
BUG=skia:1724
Review URL: https://codereview.chromium.org/1003363005
2015-03-25 04:47:02 -07:00
rmistry
f91b717a99
Ignore public API checks if COMMIT=false is in the description
...
BUG=skia:
NOTRY=true
Committed: https://skia.googlesource.com/skia/+/d1783aefcc0da86c5ff1d124c54704252d817621
Review URL: https://codereview.chromium.org/1006473002
2015-03-12 09:48:10 -07:00
mtklein
a10ea431cf
Revert of Ignore public API checks if COMMIT=false is in the description (patchset #4 id:80001 of https://codereview.chromium.org/1006473002/ )
...
Reason for revert:
I cannot upload patches:
~/skia (skskp) $ git clu --bypass-hooks
++ git log -1 --pretty=%B
+ git cl upload -t update
Using 50% similarity for rename/copy detection. Override with --similarity.
Loaded authentication cookies from /Users/mtklein/.codereview_upload_cookies
Running presubmit upload checks ...
** Presubmit ERRORS **
Since the CL is editing public API, you must have an LGTM from one of: ('reed@chromium.org', 'reed@google.com', 'bsalomon@chromium.org', 'bsalomon@google.com', 'djsollen@chromium.org', 'djsollen@google.com')
Original issue's description:
> Ignore public API checks if COMMIT=false is in the description
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d1783aefcc0da86c5ff1d124c54704252d817621
TBR=borenet@google.com ,rmistry@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1005583002
2015-03-12 09:33:46 -07:00
rmistry
d1783aefcc
Ignore public API checks if COMMIT=false is in the description
...
BUG=skia:
Review URL: https://codereview.chromium.org/1006473002
2015-03-12 08:43:22 -07:00
rmistry
01cbf6c467
Make presubmit check that #if or #ifdef does not come before includes.
...
BUG=skia:3362
Review URL: https://codereview.chromium.org/998273002
2015-03-12 07:48:40 -07:00
rmistry
896f3932c8
Automatically add NOTREECHECKS when uploading CLs from non master branches
...
These changes are possible due to the recently submitted depot_tools change
https://codereview.chromium.org/949273002/ ('Add ability to specify and run post upload hooks').
BUG=skia:
Review URL: https://codereview.chromium.org/963553002
2015-02-26 11:52:05 -08:00
rmistry
d223fb21d6
Automatically add a docs preview link and NOTRY=true when there are only docs changes.
...
These changes are possible due to the recently submitted depot_tools change
https://codereview.chromium.org/949273002/ ('Add ability to specify and run post upload hooks').
BUG=skia:3474
Review URL: https://codereview.chromium.org/960203002
2015-02-26 10:16:13 -08:00
rmistry
9407eceb27
PRESUBMIT should only check owners for the top level include directory
...
BUG=skia:1846
R=borenet@google.com
Author: rmistry@google.com
Review URL: https://codereview.chromium.org/511503002
2014-08-26 14:00:55 -07:00
rmistry
f2d83caf13
Public API changes are allowed if an owner is TBR'ed.
...
The following TBR formats are supported:
* TBR=reed
* TBR=reed@google.com
* TBR=xyz,reed
* TBR=xyz,reed@google.com
BUG=skia:2870
R=borenet@google.com
Author: rmistry@google.com
Review URL: https://codereview.chromium.org/504843003
2014-08-26 10:30:30 -07:00
rmistry
83fab4732a
Adding djsollen to list of owners
...
(SkipBuildbotRuns)
BUG=skia:
R=djsollen@google.com
Author: rmistry@google.com
Review URL: https://codereview.chromium.org/397813007
2014-07-18 05:25:57 -07:00