#2007 - Move jsdoc package inside legacy module

modules/adblockplus/manifests/legacy/webserver.pp

 # == Class: adblockplus::legacy::webserver
 #
 # A container for migrating obsolete resources in web2, formerly located
 # in manifests/webserver.pp.
 #
 # See http://hub.eyeo.com/issues/2007 for more information.
 #
 class adblockplus::legacy::webserver {
 
   class {'web::server':
     vhost => 'adblockplus.org',
     certificate => 'adblockplus.org_sslcert.pem',
     private_key => 'adblockplus.org_sslcert.key',
     is_default => true,
     aliases => ['www.adblockplus.org'],
     custom_config => template("web/adblockplus.org.conf.erb"),
     repository => 'web.adblockplus.org',
     multiplexer_locations => ['/getSubscription'],
     geoip => true,
   }
 
   ensure_packages([
     'make',
     'doxygen',
   ])
 
-  nodejs::package{'jsdoc':}

[mathias, 2017/07/20 18:30:56]

The new line before the closing bracket is considered mandatory, in order to
avoid changes to unrelated lines when a parameter is introduced at some point.
Analogous to the mandatory trailing comma and newline for each item of most
lists and hashes.

+  nodejs::package{'jsdoc':
+  }
 
   $subscription_repo = '/home/www/subscriptionlist'
 
   $fetch_repo_cmd = [
     'hg', 'clone',
     '--noupdate',
     'https://hg.adblockplus.org/subscriptionlist',
     $subscription_repo,
   ]
 
@@ skipping 32 matching lines @@
       Class['sitescripts'],
       Class['web::server'],
       Class['nodejs'],
     ],
     command => shellquote($generate_docs_cmd),
     user => www,
     minute => '5-55/10',
   }
 }