Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29589674: Issue 5942 - don't create Gecko update manifests (Closed)

Created:
Oct. 26, 2017, 8:45 p.m. by tlucas
Modified:
Nov. 17, 2017, 2:07 p.m.
CC:
Jon Sonesen
Visibility:
Public.

Description

Issue 5942 - don't create Gecko update manifests Repository: sitescripts base: 6eca5c4c3944

Patch Set 1 #

Patch Set 2 : Removing further unused code #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -107 lines) Patch
M .sitescripts.example View 2 chunks +0 lines, -2 lines 0 comments Download
M sitescripts/extensions/bin/createNightlies.py View 1 3 chunks +1 line, -17 lines 2 comments Download
M sitescripts/extensions/bin/updateUpdateManifests.py View 3 chunks +1 line, -14 lines 0 comments Download
D sitescripts/extensions/template/updates.json View 1 chunk +0 lines, -31 lines 0 comments Download
D sitescripts/extensions/test/oracle/updates.json View 1 chunk +0 lines, -40 lines 0 comments Download
M sitescripts/extensions/test/sitescripts.ini.template View 1 chunk +0 lines, -2 lines 0 comments Download
M sitescripts/extensions/test/test_updateManifests.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
tlucas
Hi everyone. This get's rid of the now redundant creation of update manifests and all ...
Oct. 26, 2017, 8:49 p.m. (2017-10-26 20:49:46 UTC) #1
Vasily Kuznetsov
It looks good to me although someone who better understands the logic should also have ...
Oct. 27, 2017, 6:47 p.m. (2017-10-27 18:47:05 UTC) #2
tlucas
Hey Vasily, there actually is some more code that can be removed (if i'm not ...
Oct. 27, 2017, 7:14 p.m. (2017-10-27 19:14:48 UTC) #3
Vasily Kuznetsov
Nov. 7, 2017, 5:49 p.m. (2017-11-07 17:49:21 UTC) #4
Then LGTM

Powered by Google App Engine
This is Rietveld