Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Issue 13896007: Devserver: run the cron over the examples. (Closed)

Created:
7 years, 8 months ago by not at google - send to devlin
Modified:
7 years, 8 months ago
Reviewers:
cduvall
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Devserver: run the cron over the examples. Do an audit of our handling of FileNotFoundErrors: make them all descriptive, and don't overcatch them. NOTRY=true BUG=226625 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=195073

Patch Set 1 #

Patch Set 2 : done #

Patch Set 3 : . #

Total comments: 2

Patch Set 4 : . #

Patch Set 5 : . #

Patch Set 6 : dont render svn files #

Patch Set 7 : try again to avoid .svn #

Unified diffs Side-by-side diffs Delta from patch set Stats (+99 lines, -98 lines) Patch
M chrome/common/extensions/docs/server2/api_data_source.py View 2 chunks +3 lines, -7 lines 0 comments Download
M chrome/common/extensions/docs/server2/api_list_data_source.py View 2 chunks +1 line, -5 lines 0 comments Download
M chrome/common/extensions/docs/server2/caching_file_system.py View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/common/extensions/docs/server2/example_zipper.py View 1 2 2 chunks +11 lines, -6 lines 0 comments Download
M chrome/common/extensions/docs/server2/example_zipper_test.py View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/extensions/docs/server2/fake_fetchers.py View 2 chunks +6 lines, -6 lines 0 comments Download
M chrome/common/extensions/docs/server2/handler.py View 1 3 chunks +23 lines, -9 lines 0 comments Download
M chrome/common/extensions/docs/server2/integration_test.py View 1 2 3 4 5 6 2 chunks +6 lines, -3 lines 0 comments Download
M chrome/common/extensions/docs/server2/local_file_system.py View 3 chunks +12 lines, -12 lines 0 comments Download
M chrome/common/extensions/docs/server2/offline_file_system.py View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/common/extensions/docs/server2/server_instance.py View 1 3 chunks +20 lines, -29 lines 0 comments Download
M chrome/common/extensions/docs/server2/sidenav_data_source.py View 2 chunks +4 lines, -8 lines 0 comments Download
M chrome/common/extensions/docs/server2/subversion_file_system.py View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/common/extensions/docs/server2/template_data_source.py View 1 2 3 3 chunks +5 lines, -5 lines 0 comments Download

Messages

Total messages: 12 (0 generated)
not at google - send to devlin
The last patch wasn't quite enough...
7 years, 8 months ago (2013-04-18 23:24:54 UTC) #1
cduvall
lgtm https://codereview.chromium.org/13896007/diff/5001/chrome/common/extensions/docs/server2/template_data_source.py File chrome/common/extensions/docs/server2/template_data_source.py (right): https://codereview.chromium.org/13896007/diff/5001/chrome/common/extensions/docs/server2/template_data_source.py#newcode149 chrome/common/extensions/docs/server2/template_data_source.py:149: os.path.join(base_path, FormatKey(template_name))) might want to use '/'.join since ...
7 years, 8 months ago (2013-04-18 23:31:15 UTC) #2
not at google - send to devlin
https://codereview.chromium.org/13896007/diff/5001/chrome/common/extensions/docs/server2/template_data_source.py File chrome/common/extensions/docs/server2/template_data_source.py (right): https://codereview.chromium.org/13896007/diff/5001/chrome/common/extensions/docs/server2/template_data_source.py#newcode149 chrome/common/extensions/docs/server2/template_data_source.py:149: os.path.join(base_path, FormatKey(template_name))) On 2013/04/18 23:31:16, cduvall wrote: > might ...
7 years, 8 months ago (2013-04-18 23:36:36 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/13896007/5004
7 years, 8 months ago (2013-04-19 02:48:17 UTC) #4
commit-bot: I haz the power
Presubmit check for 13896007-5004 failed and returned exit status 1. INFO:root:Found 13 file(s). ERROR:root:extensions/examples/api/downloads/download_links/icon.png had ...
7 years, 8 months ago (2013-04-19 02:49:32 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/13896007/12001
7 years, 8 months ago (2013-04-19 03:08:09 UTC) #6
commit-bot: I haz the power
Presubmit check for 13896007-12001 failed and returned exit status 1. INFO:root:Found 14 file(s). ERROR:root:extensions/examples/api/downloads/download_links/icon.png had ...
7 years, 8 months ago (2013-04-19 03:09:08 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/13896007/14001
7 years, 8 months ago (2013-04-19 03:16:54 UTC) #8
commit-bot: I haz the power
Presubmit check for 13896007-14001 failed and returned exit status 1. INFO:root:Found 14 file(s). ERROR:root:extensions/examples/api/downloads/download_links/icon.png had ...
7 years, 8 months ago (2013-04-19 03:17:52 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/13896007/2002
7 years, 8 months ago (2013-04-19 03:30:47 UTC) #10
commit-bot: I haz the power
Presubmit check for 13896007-2002 failed and returned exit status 1. INFO:root:Found 14 file(s). ERROR:root:extensions/examples/api/downloads/download_links/icon.png had ...
7 years, 8 months ago (2013-04-19 03:31:51 UTC) #11
not at google - send to devlin
7 years, 8 months ago (2013-04-19 03:35:33 UTC) #12
Message was sent while issue was closed.
Committed patchset #7 manually as r195073.

Powered by Google App Engine
This is Rietveld 408576698