Noissue - Restructure Build-and-Release-Tools and Sitescripts modules

pages/modules.html

Index: pages/modules.html
===================================================================
--- a/pages/modules.html
+++ b/pages/modules.html
@@ -135,21 +135,18 @@
   </tr>
 </table>
 
-<table id="build-and-release-tools">
+<table id="tooling">
   <tr>
     <th>{{table-header-name Name}}</th>
-    <td>Build and Release Tools</td>
+    <td>Tooling</td>

[Sebastian Noack, 2017/10/12 03:20:36]

This name isn't final, but rather a working title. Other options that come to
mind would be "Automation" or "Tools". What do you think?

[kzar, 2017/10/12 09:10:48]

I don't like this name much, but I can't think of anything better right now.

[tlucas, 2017/10/13 16:35:42]

FWIW, i personally prefer the "Automation" suggestion.

[Sebastian Noack, 2017/10/13 20:01:56]

Done.

   </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>

[Sebastian Noack, 2017/10/12 03:20:36]

Dave, let me know if you would want to be a module owner peer on the "Tooling".

[kzar, 2017/10/12 09:10:48]

I don't mind, I think I can still add something but on the other hand I wouldn't
be offended if you removed me. I'm hardly doing much Python these days, but on
the other hand I do have a good idea about where our tooling could be improved.

[Sebastian Noack, 2017/10/12 22:28:09]

If you want to request improvements, just file issues (regardless whether you
are module owner/peer or not). However, if you are module owner peer you have to
invest some reviewing patches and triaging issues (not only those you requested
yourself). But I leave it up to you.

-  </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 +160,13 @@
           <a href="https://hg.adblockplus.org/buildtools">buildtools</a>
         </li>
         <li>
-          <a href="https://hg.adblockplus.org/screenshotter">screenshotter</a>

[Sebastian Noack, 2017/10/12 03:20:36]

I have no idea what this thing was, but apparently it was a legacy Gecko
extension, so we can just deprecate it, now. :)

+          <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 +479,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 +493,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="#tooling"><fix>Tooling</fix></a>)}}

[Vasily Kuznetsov, 2017/10/12 12:14:35]

Redirecting to Tooling to understand which part of sitescripts is not under this
module seems inconvenient for the reader. How about we just say "except
sitescripts/extensions/*", same as in Tooling? This would be duplication, of
course, but in this case readability seems more important.

[Sebastian Noack, 2017/10/12 22:28:09]

I did it consistently with how Platform and User Interface are doing it. Since
there are several paths there, duplicating them inline isn't an option, and I'm
not sure whether it is worth to introduce an inconsistency here.

[Vasily Kuznetsov, 2017/10/13 09:44:01]

Acknowledged.

         </li>
       </ul>
     </td>