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

Unified Diff: pages/modules.html

Issue 29573947: Noissue - Restructure Build-and-Release-Tools and Sitescripts modules (Closed)
Patch Set: Added description Created Oct. 20, 2017, 1:51 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pages/modules.html
===================================================================
--- a/pages/modules.html
+++ b/pages/modules.html
@@ -65,7 +65,7 @@
</tr>
<tr>
<th>{{table-header-description Description}}</th>
- <td>{{module-ui-description The shared UI code.}}</td>
+ <td>{{module-ui-description The User Interface of the Adblock Plus browser extension.}}</td>
Sebastian Noack 2017/10/20 01:53:38 While on it, I also fixed this description, since
kzar 2017/10/20 11:48:13 Fair enough, but Thomas should check this change t
Thomas Greiner 2017/10/20 14:38:12 I'm ok with this change.
</tr>
<tr>
<th>{{table-header-owner Owner}}</th>
@@ -135,21 +135,22 @@
</tr>
</table>
-<table id="build-and-release-tools">
+<table id="automation">
<tr>
<th>{{table-header-name Name}}</th>
- <td>Build and Release Tools</td>
+ <td>Automation</td>
+ </tr>
+ <tr>
+ <th>{{table-header-description Description}}</th>
+ <td>{{module-automation-description The build and release automation for the Adblock Plus browser extension, and some general coding tools.}}</td>
</tr>
<tr>
<th>{{table-header-owner Owner}}</th>
<td><a href="mailto:sebastian@adblockplus.org">Sebastian Noack</a></td>
</tr>
<tr>
- <th rowspan="2">{{table-header-peers Peer(s)}}</th>
- <td><a href="mailto:dave@adblockplus.org">Dave Barker</a></td>
- </tr>
- <tr>
- <td><a href="mailto:vasily@adblockplus.org">Vasily Kuznetsov</a></td>
+ <th>{{table-header-peers Peer(s)}}</th>
+ <td><a href="mailto:tristan@adblockplus.org">Tristan Lucas</a></td>
</tr>
<tr>
<th>{{table-header-super-reviewer Super reviewer}}</th>
@@ -163,7 +164,13 @@
<a href="https://hg.adblockplus.org/buildtools">buildtools</a>
</li>
<li>
- <a href="https://hg.adblockplus.org/screenshotter">screenshotter</a>
+ <a href="https://hg.adblockplus.org/codingtools">codingtools</a>
+ </li>
+ <li>
+ <a href="https://hg.adblockplus.org/sitescripts">sitescripts</a>
+ <ul>
+ <li>sitescripts/extensions/*</li>
+ </ul>
</li>
</ul>
</td>
@@ -476,10 +483,6 @@
<td><a href="mailto:vasily@adblockplus.org">Vasily Kuznetsov</a></td>
</tr>
<tr>
- <th>{{table-header-peers Peer(s)}}</th>
- <td><a href="mailto:sebastian@adblockplus.org">Sebastian Noack</a></td>
- </tr>
- <tr>
<th>{{table-header-super-reviewer Super reviewer}}</th>
<td><a href="mailto:felix@adblockplus.org">Felix Dahlke</a></td>
</tr>
@@ -494,13 +497,10 @@
<a href="https://hg.adblockplus.org/cms">cms</a>
</li>
<li>
- <a href="https://hg.adblockplus.org/codingtools">codingtools</a>
- </li>
- <li>
<a href="https://hg.adblockplus.org/python-abp">python-abp</a>
</li>
<li>
- <a href="https://hg.adblockplus.org/sitescripts">sitescripts</a>
+ {{module-sitescripts-source <a href="https://hg.adblockplus.org/sitescripts"><fix>sitescripts</fix></a> (except <a href="#automation"><fix>Automation</fix></a>)}}
</li>
</ul>
</td>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld