aboutsummaryrefslogtreecommitdiffhomepage
path: root/PRESUBMIT.py
Commit message (Collapse)AuthorAge
* PRESUBMIT should only check owners for the top level include directoryGravatar rmistry2014-08-26
| | | | | | | | | BUG=skia:1846 R=borenet@google.com Author: rmistry@google.com Review URL: https://codereview.chromium.org/511503002
* Public API changes are allowed if an owner is TBR'ed.Gravatar rmistry2014-08-26
| | | | | | | | | | | | | | | 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
* Adding djsollen to list of ownersGravatar rmistry2014-07-18
| | | | | | | | | | | (SkipBuildbotRuns) BUG=skia: R=djsollen@google.com Author: rmistry@google.com Review URL: https://codereview.chromium.org/397813007
* Use new common tools in Python scriptsGravatar Eric Boren2014-06-25
| | | | | | | BUG=skia:2682 R=rmistry@google.com Review URL: https://codereview.chromium.org/330423004
* Check if the issue owner is in the AUTHORS file in PRESUBMIT.pyGravatar commit-bot@chromium.org2014-02-03
| | | | | | | | | | | | | | BUG=skia:2072 NOTRY=true (SkipBuildbotRuns) R=borenet@google.com, hcm@google.com Author: rmistry@google.com Review URL: https://codereview.chromium.org/131293005 git-svn-id: http://skia.googlecode.com/svn/trunk@13277 2bbb7eff-a529-9590-31e7-b0007b416f81
* Do not enforce the public API presubmit check for Revert CLs.Gravatar commit-bot@chromium.org2014-01-31
| | | | | | | | | | | | | NOTRY=true BUG=skia:2068 R=borenet@google.com Author: rmistry@google.com Review URL: https://codereview.chromium.org/151573002 git-svn-id: http://skia.googlecode.com/svn/trunk@13270 2bbb7eff-a529-9590-31e7-b0007b416f81
* Adding rmistry for emergency reverts.Gravatar commit-bot@chromium.org2014-01-28
| | | | | | | | | | | | | | | (SkipBuildbotRuns) TBR= NOTRY=true NOTREECHECKS=true BUG=skia: Author: rmistry@google.com Review URL: https://codereview.chromium.org/148583005 git-svn-id: http://skia.googlecode.com/svn/trunk@13212 2bbb7eff-a529-9590-31e7-b0007b416f81
* If any change is made to the public API then make sure there is an LGTM from ↵Gravatar rmistry@google.com2013-08-12
| | | | | | | | | | | | | | | | | | | an owner. This feature has been requested by reed@. If no LGTM is found from an owner then the following msg is displayed: ** 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') (SkipBuildbotRuns) R=borenet@google.com Review URL: https://codereview.chromium.org/22453004 git-svn-id: http://skia.googlecode.com/svn/trunk@10676 2bbb7eff-a529-9590-31e7-b0007b416f81
* If the tree is closed then print information about contacting the sheriff.Gravatar rmistry@google.com2013-04-12
| | | | | | | | | | | The motivation for this CL is because it is not very obvious on how to submit when tree is closed in git. Also added a section 'How to submit when tree is closed' in https://sites.google.com/site/skiadocs/developer-documentation/tree-sheriff (SkipBuildbotRuns) Review URL: https://codereview.chromium.org/13932026 git-svn-id: http://skia.googlecode.com/svn/trunk@8666 2bbb7eff-a529-9590-31e7-b0007b416f81
* Display prompt for caution only if it is an interactive shell.Gravatar rmistry@google.com2013-03-29
| | | | | | | | | Created to fix https://code.google.com/p/skia/issues/detail?id=1197 : Skia Commit Queue quits if the tree is in Caution. (SkipBuildbotRuns) Review URL: https://codereview.chromium.org/13325002 git-svn-id: http://skia.googlecode.com/svn/trunk@8446 2bbb7eff-a529-9590-31e7-b0007b416f81
* Presubmit should check that source files end in atleast one newlineGravatar rmistry@google.com2013-01-25
| | | | | | Review URL: https://codereview.appspot.com/7193063 git-svn-id: http://skia.googlecode.com/svn/trunk@7399 2bbb7eff-a529-9590-31e7-b0007b416f81
* Display a warning prompt in presubmit if the tree status is in 'Caution'.Gravatar rmistry@google.com2013-01-23
| | | | | | | | | | | | | | | | | | | | Example presubmit output: Running presubmit upload checks ... ** Presubmit Warnings ** Tree state is: caution *************** Please use caution in checkins http://skia-tree-status-staging.appspot.com/banner-status?format=json *************** There were presubmit warnings. Are you sure you wish to continue? (y/N): Review URL: https://codereview.appspot.com/7200043 git-svn-id: http://skia.googlecode.com/svn/trunk@7337 2bbb7eff-a529-9590-31e7-b0007b416f81
* Adding CheckChangeHasOnlyOneEol to upload and commit presubmit checksGravatar rmistry@google.com2013-01-17
| | | | | | Review URL: https://codereview.appspot.com/7151043 git-svn-id: http://skia.googlecode.com/svn/trunk@7247 2bbb7eff-a529-9590-31e7-b0007b416f81
* Top-level presubmit script for Skia trunk that checks ↵Gravatar rmistry@google.com2013-01-17
https://skia-tree-status.appspot.com/banner-status?format=json before a CL is committed. gcl and git-cl will check for and run the presubmit script before you upload and/or commit your changes. Review URL: https://codereview.appspot.com/7138046 git-svn-id: http://skia.googlecode.com/svn/trunk@7242 2bbb7eff-a529-9590-31e7-b0007b416f81