andymeneely / chromium-history

Scripts and data related Chromium's history
11 stars 4 forks source link

Clean up build output #212

Closed andymeneely closed 9 years ago

andymeneely commented 9 years ago

Currently our build output has a cryptic error message with a bunch of numbers when the code reviews get parsed. Make this a real error message if it's wrong, or remove this output.

Parsing JSON Code Reviews
6714032|4278

10383257|1

10913048|4012

10916327|1

10916327|918

19267003|30002

24253006|11001

90243002|20001

90243002|40001

83593004|200001

89063003|60001

89063003|80001

83643002|1

83643002|70001

83643002|240001

89533002|1

89243002|20001

89023003|20001

77453007|260001

77453007|280001

77453007|300001

46573002|1

46573002|120001

48773005|1

48773005|390002

48773005|780001

46043016|70001

112973008|10001

108513011|1

108513011|130001

140993006|390007

140993006|220034

182313010|1

247123002|20001

247123002|70001
Felivel commented 9 years ago

I have removed this output, as we already know these ids. All we need to do know is use this list to reparse these code reviews.