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

Issue 29326213: Issue 3039 - Generate qunit/index.html based on metadata (Closed)

Created:
Sept. 9, 2015, 5:48 p.m. by kzar
Modified:
Sept. 9, 2015, 6:22 p.m.
Reviewers:
Sebastian Noack
Visibility:
Public.

Description

Issue 3039 - Generate qunit/index.html based on metadata

Patch Set 1 #

Total comments: 3

Patch Set 2 : Remove JQuery which we're not using and qunit.js which is now hard coded into the template #

Patch Set 3 : Updated buildtools dependency again #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -54 lines) Patch
M dependencies View 1 2 1 chunk +1 line, -1 line 0 comments Download
M metadata.common View 1 1 chunk +18 lines, -0 lines 0 comments Download
D qunit/index.html View 1 chunk +0 lines, -53 lines 0 comments Download

Messages

Total messages: 7
kzar
Patch Set 1
Sept. 9, 2015, 5:49 p.m. (2015-09-09 17:49:03 UTC) #1
Sebastian Noack
https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common File metadata.common (right): https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common#newcode18 metadata.common:18: qunit.js Argh. I think this should be hard-coded in ...
Sept. 9, 2015, 5:56 p.m. (2015-09-09 17:56:51 UTC) #2
kzar
https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common File metadata.common (right): https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common#newcode18 metadata.common:18: qunit.js On 2015/09/09 17:56:51, Sebastian Noack wrote: > Argh. ...
Sept. 9, 2015, 6:01 p.m. (2015-09-09 18:01:00 UTC) #3
Sebastian Noack
https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common File metadata.common (right): https://codereview.adblockplus.org/29326213/diff/29326214/metadata.common#newcode18 metadata.common:18: qunit.js On 2015/09/09 18:01:00, kzar wrote: > On 2015/09/09 ...
Sept. 9, 2015, 6:08 p.m. (2015-09-09 18:08:37 UTC) #4
Sebastian Noack
LGTM
Sept. 9, 2015, 6:08 p.m. (2015-09-09 18:08:55 UTC) #5
kzar
Patch Set 3 : Updated buildtools dependency again
Sept. 9, 2015, 6:16 p.m. (2015-09-09 18:16:44 UTC) #6
Sebastian Noack
Sept. 9, 2015, 6:18 p.m. (2015-09-09 18:18:38 UTC) #7
LGTM

Powered by Google App Engine
This is Rietveld