aboutsummaryrefslogtreecommitdiffhomepage
path: root/PRESUBMIT.py
Commit message (Collapse)AuthorAge
* Update CQ extra trybots after switch to DebianGravatar Eric Boren2017-06-29
| | | | | | | | Bug: skia: Change-Id: I6611638097b473d719106c239012b5382d962941 Reviewed-on: https://skia-review.googlesource.com/21266 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Use Gerrit style format for footers in PresubmitGravatar Ravi Mistry2017-05-23
| | | | | | | | | | | This also fixes a bug where CQ keywords were added multiple times because it was looking for them in the wrong place (new_description_lines instead of the footers). Bug: skia:6667 Change-Id: I91232297343145884b3fe7c2e923373679cf029e Reviewed-on: https://skia-review.googlesource.com/17704 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Separate lines added via post upload from original descriptionGravatar Ravi Mistry2017-05-08
| | | | | | | | | | BUG=skia:6598 NOTRY=true Change-Id: I23fa415788e91da01d33ab3ab912aa420636574c Reviewed-on: https://skia-review.googlesource.com/15869 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Revert "Presubmit: ensure empty line between CL title and description"Gravatar Ravi Mistry2017-05-08
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9518830650a45446756d7b621fc87e7a8ff52476. Reason for revert: <INSERT REASONING HERE> Original change's description: > Presubmit: ensure empty line between CL title and description > > Bug: skia: > Change-Id: If1b2d655e9be4b960f76d2d17ce1cf1126f7b2d4 > Reviewed-on: https://skia-review.googlesource.com/15624 > Reviewed-by: Eric Boren <borenet@google.com> > Commit-Queue: Eric Boren <borenet@google.com> > TBR=borenet@google.com,rmistry@google.com NOTRY=true # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I311ee790f152dbb8325214650838839f6821c995 Reviewed-on: https://skia-review.googlesource.com/15862 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Presubmit: ensure empty line between CL title and descriptionGravatar Eric Boren2017-05-05
| | | | | | | | Bug: skia: Change-Id: If1b2d655e9be4b960f76d2d17ce1cf1126f7b2d4 Reviewed-on: https://skia-review.googlesource.com/15624 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Change PRESUBMIT.py to use [Get|Update]DescriptionLinesGravatar Eric Boren2017-04-05
| | | | | | | | | This should fix the "git cl upload" failures on the recipe roller. Change-Id: I1c433a5a6dc7f52034bef8b2178b3041b8f4661e Reviewed-on: https://skia-review.googlesource.com/11020 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Clean up more references to GYP.Gravatar Mike Klein2017-02-06
| | | | | | | | | | Delete files only used by GYP, and files that used GYP. Neither can possibly be actively used. Beyond that, just a couple doc tweaks. Change-Id: I0220d7226e7bb9ed7c54a7d8f2906a718313c521 Reviewed-on: https://skia-review.googlesource.com/8062 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Hal Canary <halcanary@google.com>
* Add hcm@ to public API ownersGravatar Ravi Mistry2017-01-19
| | | | | | | | | | 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>
* Do not use post upload hooks for 'GOLD_TRYBOT_URL'Gravatar Ravi Mistry2016-11-23
| | | | | | | | | | | | | 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>
* [nobuildbot] Migrate all Linux GCE botsGravatar Eric Boren2016-11-23
| | | | | | | | | | | | 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>
* Split SkSL and utils apart again into .gni files.Gravatar Mike Klein2016-10-26
| | | | | | | | | | | | | 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>
* Add infra_tests.py, recipe, buildbotless botGravatar borenet2016-10-14
| | | | | | | | | 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
* Fix GN formatting presubmit on Windows.Gravatar Mike Klein2016-10-11
| | | | | | | | | | | | 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>
* Fix public API msg in PRESUBMIT.pyGravatar Ravi Mistry2016-10-05
| | | | | | | | | | | 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>
* PRESUBMIT.py improvementsGravatar Ravi Mistry2016-10-05
| | | | | | | | | | | | | | * 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>
* Add --test to gen_tasks.go, add presubmit checkGravatar borenet2016-10-04
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2392083002 Review-Url: https://codereview.chromium.org/2392083002
* To prep for removing CMake bots, take them off the CQ and presubmit.Gravatar mtklein2016-09-19
| | | | | | | | | | | | | | | | | | 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
* Fix PostUploadHook for Gerrit issuesGravatar rmistry2016-09-01
| | | | | | | | | | | 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
* Enable Gerrit support in Skia's PostUploadHookGravatar rmistry2016-09-01
| | | | | | | | BUG=skia:5676 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2299203002 NOTRY=true Review-Url: https://codereview.chromium.org/2299203002
* Use methods from depot_tool's Changelist object instead of Rietveld RPCGravatar rmistry2016-09-01
| | | | | | | | | | 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
* Fix up presubmit for SAN GCC->Clang fix.Gravatar mtklein2016-08-31
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2302553002 Review-Url: https://codereview.chromium.org/2302553002
* Add Gerrit API support to Skia's PRESUBMIT.pyGravatar rmistry2016-08-29
| | | | | | | | | 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
* Roll GN: gn format is --in-place by defaultGravatar mtklein2016-08-10
| | | | | | | | | 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
* Require gn format in presubmitGravatar mtklein2016-08-03
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2203283003 Review-Url: https://codereview.chromium.org/2203283003
* Only run recipe simulation tests if infra/ changes.Gravatar mtklein2016-07-27
| | | | | | | | | | 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
* Move Skia recipes from build repoGravatar borenet2016-07-26
| | | | | | | BUG=skia:5563 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2175373002 Review-Url: https://codereview.chromium.org/2175373002
* Specifying master names with "master." prefix is deprecatedGravatar rmistry2016-07-13
| | | | | | | | NOTRY=true BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2151623002 Review-Url: https://codereview.chromium.org/2151623002
* Do not use the deprecated CQ_EXTRA_TRYBOTSGravatar rmistry2016-07-11
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135063003 Review-Url: https://codereview.chromium.org/2135063003
* Fix Gold trybot results URLGravatar rmistry2016-05-20
| | | | | | | BUG=skia:5324 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=1996103002 Review-Url: https://codereview.chromium.org/1996103002
* 10.9 is no moreGravatar mtklein2016-02-25
| | | | | | | 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
* Port Skia recipe to normal Python scripts, move to Skia repoGravatar borenet2016-02-18
| | | | | | | 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
* Add link to Gold trybot runs in every uploaded Skia CLGravatar rmistry2015-12-15
| | | | | | | 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
* Comments Style: s/skbug.com/bug.skia.org/Gravatar halcanary2015-11-07
| | | | | | DOCS_PREVIEW= https://skia.org/?cl=1432503003 Review URL: https://codereview.chromium.org/1432503003
* Automatically add TSAN bots to the CQ for changes affecting SkAtomics.hGravatar mtklein2015-10-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1396523002
* Automatically add extra CQ trybots for predetermined pathsGravatar rmistry2015-10-01
| | | | | | | | | | | | | | 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
* Resolve TODOs in PRESUBMIT.pyGravatar rmistry2015-10-01
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1384453004
* update presubmit to know about include/privateGravatar mtklein2015-07-28
| | | | | | BUG=skia:4126 Review URL: https://codereview.chromium.org/1259953004
* Revert "Move headers used by headers in include/ to include/private."Gravatar Mike Klein2015-07-01
| | | | | | | | This reverts commit 928e16565f3e69cd6661e9ad300ac17e3f33c0c8. BUG=skia: Review URL: https://codereview.chromium.org/1213093004.
* Move headers used by headers in include/ to include/private.Gravatar mtklein2015-07-01
| | | | | | | | | | | 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
* Revert "Move headers used by headers in include/ to include/private."Gravatar jvanverth2015-07-01
| | | | | | | | | | | | | This reverts commit a89f55198bdc58f0b6f6196907ab25a6afc1a661. Reason: breaking the roll. TBR=mtklein@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1216033008
* Move headers used by headers in include/ to include/private.Gravatar mtklein2015-07-01
| | | | | | | | | 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
* Presubmit: add PRESUBMIT=false to non-master branch changesGravatar borenet2015-06-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1161973005
* Add presubmit to run `{dm,nanobench}_flags.py test` when changed.Gravatar mtklein2015-05-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1129443005
* Bypass owners check for dry run CLs.Gravatar rmistry2015-04-07
| | | | | | | BUG=skia: NOTRY=true Review URL: https://codereview.chromium.org/1068853002
* Find the HASHTAGS file even if you are not at the checkout root.Gravatar rmistry2015-04-01
| | | | | | | | 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
* Add post upload hook to substitute hashtags for their mapped textGravatar rmistry2015-03-25
| | | | | | | BUG=skia:3586 NOTRY=true Review URL: https://codereview.chromium.org/1004733009
* More file types to check for (C).Gravatar mtklein2015-03-25
| | | | | | | | | | | | - objective C++ (31) - objective C (4) - C (2) - alternate C++ extension (1) - Go (1) BUG=skia: Review URL: https://codereview.chromium.org/1032143002
* Add copyright check to PRESUBMITGravatar borenet2015-03-25
| | | | | | BUG=skia:1724 Review URL: https://codereview.chromium.org/1003363005
* Ignore public API checks if COMMIT=false is in the descriptionGravatar rmistry2015-03-12
| | | | | | | | | BUG=skia: NOTRY=true Committed: https://skia.googlesource.com/skia/+/d1783aefcc0da86c5ff1d124c54704252d817621 Review URL: https://codereview.chromium.org/1006473002
* Revert of Ignore public API checks if COMMIT=false is in the description ↵Gravatar mtklein2015-03-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (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