30 Commits

Autor SHA1 Mensagem Data
maruel@chromium.org fc7ffe91e6 Enforce presubmit check of try run. Add a nice message to give feedback to the right person.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/155354

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20397 0039d316-1c4b-4281-b951-d872f2087c98
2009-07-10 19:09:08 +00:00
maruel@chromium.org 69f5bf622d Enforce correct mime-type on some files formats (especially images).
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/147221

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19437 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-26 23:00:37 +00:00
maruel@chromium.org c4df07316e Display the actual tree status when the tree is closed. Much more helpful.
TEST=Try committing when the tree is closed to see.
BUG=none
Review URL: http://codereview.chromium.org/119437

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18125 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-11 00:41:32 +00:00
maruel@chromium.org 8dfe256271 Use new canned checks in PRESUBMIT.py
Also fix gcl presubmit tree status check when the change hasn't been uploaded nor gcl try'ed.

Remove PRESUBMIT_unittest.py since it's not useful anymore.

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/119432

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18124 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-11 00:33:40 +00:00
maruel@chromium.org 3d82f6d137 Use input_api.ReadFile() instead of rolling out its own.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/118484

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18051 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-10 13:39:20 +00:00
nsylvain@chromium.org f22a3a7e73 Dummy change to trigger the buildbots
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18019 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-10 02:39:31 +00:00
maruel@chromium.org df9a400c1d Add try job status check on commit.
It acts as an advisory presubmit check for now since the try server isn't that stable.

BUG=none
TEST=gcl commit warns the user when a pending change hasn't passed the try job.
Review URL: http://codereview.chromium.org/119295

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17870 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-08 17:31:51 +00:00
nsylvain@chromium.org 78ef7b2576 Dummy change to trigger the slaves - NO CODE CHANGE
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17846 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-07 19:24:51 +00:00
nsylvain@chromium.org 7b1d595c5f Dummy change to trigger the build slaves - NO CODE CHANGE
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17845 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-07 17:43:16 +00:00
maruel@chromium.org 4bc8473532 Add the tree check there instead of implementing it in the root PRESUBMIT.py.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/119201

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17729 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-05 14:25:54 +00:00
maruel@chromium.org 09546e0012 Fix PRESUBMIT.py's path handling.
Review URL: http://codereview.chromium.org/118202

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17574 0039d316-1c4b-4281-b951-d872f2087c98
2009-06-04 00:00:42 +00:00
kuchhal@chromium.org d0c1719670 Revert "Revert "Add presubmit checks to look for BUG= and TEST= tags in change list description.""
This reverts commit fb3644d302ea26efbe2a2e761beea2d3f26bac59.

Review URL: http://codereview.chromium.org/115577


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16612 0039d316-1c4b-4281-b951-d872f2087c98
2009-05-21 17:04:56 +00:00
kuchhal@chromium.org 1fd26f45e3 Revert "Add presubmit checks to look for BUG= and TEST= tags in change list description."
Review URL: http://codereview.chromium.org/113309


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15909 0039d316-1c4b-4281-b951-d872f2087c98
2009-05-12 22:26:55 +00:00
kuchhal@chromium.org f92604103b Add presubmit checks to look for BUG= and TEST= tags in change list description (Part 2 of 2).
BUG=11287
TEST=gcl commit should fail unless BUG= & TEST= exist in change list description or --no_presubmit is used.

Review URL: http://codereview.chromium.org/115261


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15906 0039d316-1c4b-4281-b951-d872f2087c98
2009-05-12 21:58:31 +00:00
wtc@chromium.org ec46828070 Dummy checkin to trigger the bots after the build error
"Chromium XP :archived build failed".

TBR=nsylvain
Review URL: http://codereview.chromium.org/99034

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14556 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-26 01:36:48 +00:00
nsylvain@chromium.org 7a585dbfea Dummy checkin to trigger the bots after the master factory refactoring - part 4
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14536 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-25 20:54:42 +00:00
nsylvain@chromium.org 273d005cd4 Dummy checkin to trigger the bots after the master factory refactoring - part 3
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14535 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-25 20:12:15 +00:00
nsylvain@chromium.org 6efcb26589 Dummy checkin to trigger the bots after the master factory refactoring
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14532 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-25 19:41:20 +00:00
nsylvain@chromium.org 8e0aaaf1bc Dummy checkin to trigger the bots after the master factory refactoring
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14531 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-25 19:30:50 +00:00
maruel@chromium.org bedaf28105 REmove one unneeded seam.
Review URL: http://codereview.chromium.org/69019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13853 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-16 17:54:07 +00:00
sgk@google.com 37246927d2 Remove the 'Hammer' name in favor of the Debug|Release names
and paths we'll use in gyp-generated files.
Review URL: http://codereview.chromium.org/56158

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12975 0039d316-1c4b-4281-b951-d872f2087c98
2009-04-01 20:13:04 +00:00
sgk@google.com 84d8a992cb Revert r12849 (conversion of "Hammer" names to "Debug").
Review URL: http://codereview.chromium.org/57051

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12854 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-31 11:34:03 +00:00
sgk@google.com 887f387549 Replace "Hammer" references with "Debug" as appropriate.
Review URL: http://codereview.chromium.org/53118

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12849 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-31 08:37:26 +00:00
nsylvain@chromium.org 10d6c21e4a Fix style in Presubmit.py
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11715 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-15 19:45:53 +00:00
thestig@chromium.org f00998c1b1 Allow PRESUBMIT.py to handle deleted files.
This is a follow up to Issue 43098.
Review URL: http://codereview.chromium.org/42086

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11619 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-13 04:35:37 +00:00
maruel@chromium.org 0563bf24f9 Add exceptions to max_cols rule.
Review URL: http://codereview.chromium.org/42019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11340 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-10 17:13:42 +00:00
deanm@chromium.org bb46ceedc3 Add end of file newline checks to PRESUBMIT.py.
This makes sure that:
  - Files end in a newline character, this is otherwise a fatal error with GCC.
  - Files end in only one newline character, and not a bunch of whitespace.

Review URL: http://codereview.chromium.org/43017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11330 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-10 13:55:09 +00:00
maruel@chromium.org f10cf0e7a0 Temporarily disable max_cols=80 rule since there's too many source files that trigger this rule at the moment.
Review URL: http://codereview.chromium.org/39192

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10992 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-05 16:14:55 +00:00
maruel@chromium.org b12d14d967 - Add «no trailing whitespace» and <=80 cols rules.
- Remove the invalid import os statement (PRESUBMIT.py shouldn't import anything).
- Add excluded paths. Mostly third parties.
- Increase what is considered a source file.
- Don't check for DO NOT SUBMIT on upload.
Review URL: http://codereview.chromium.org/28219

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10983 0039d316-1c4b-4281-b951-d872f2087c98
2009-03-05 14:03:14 +00:00
maruel@chromium.org c8c16a1386 Very basic starter presubmit script for Chromium, as requested by maruel.
At present this checks the following:
- No tabs in source files
- The words "DO-NOT-SUBMIT" (without the dashes) do not appear in the CL
  description or in files in the change
- No CRLF in source files (.cc, .h, .mm, .py)

Limitations:
- The general limitations of the current presubmit API implementation
  apply.  One of these is that gcl only walks up to the checkout root
  searching for presubmit files, so if you've mapped stuff into your
  client using DEPS, it will only walk up to where the root of that
  mapping.

Patch contributed by Jói.

Review: http://codereview.chromium.org/20474


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10015 0039d316-1c4b-4281-b951-d872f2087c98
2009-02-19 16:33:12 +00:00