Bug 31515 - Add bugzilla-tool check-style
Summary: Add bugzilla-tool check-style
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 31422
  Show dependency tree / graph
 
Reported: 2009-11-14 18:42 PST by Adam Barth
Modified: 2009-11-15 18:30 PST (History)
2 users (show)

See Also:

Attachments
Patch (4.58 KB, patch)
2009-11-14 18:51 PST, Adam Barth 		no flags Details | Formatted Diff | Diff
Patch (5.86 KB, patch)
2009-11-14 22:09 PST, Adam Barth 		no flags Details | Formatted Diff | Diff
Patch (5.99 KB, patch)
2009-11-14 22:28 PST, Adam Barth 		no flags Details | Formatted Diff | Diff
Patch (4.71 KB, patch)
2009-11-14 23:08 PST, Adam Barth 		no flags Details | Formatted Diff | Diff
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2009-11-14 18:42:26 PST 
This will be used by the try-queue.
Comment 1 Adam Barth 2009-11-14 18:51:49 PST 
Created attachment 43237 [details]
Patch
Comment 2 Adam Barth 2009-11-14 22:09:03 PST 
Created attachment 43242 [details]
Patch
Comment 3 Adam Barth 2009-11-14 22:28:28 PST 
Created attachment 43243 [details]
Patch
Comment 4 Adam Barth 2009-11-14 23:08:42 PST 
Created attachment 43244 [details]
Patch
Comment 5 Darin Adler 2009-11-15 15:21:48 PST 
Comment on attachment 43244 [details]
Patch

r=me
Comment 6 Adam Barth 2009-11-15 15:24:43 PST 
Comment on attachment 43244 [details]
Patch

Clearing flags on attachment: 43244

Committed r51001: <http://trac.webkit.org/changeset/51001>
Comment 7 Adam Barth 2009-11-15 15:24:49 PST 
All reviewed patches have been landed.  Closing bug.
Comment 8 Eric Seidel (no email) 2009-11-15 18:30:51 PST 
Comment on attachment 43244 [details]
Patch

I don't like that this applies the patches to check their style.  That seems unnecessary.  I also don't like that this has no tests. :(