fix up some comments

Author: Alistair Laing

lib/extensions/extensions.js

@@ -9,23 +9,24 @@
  *    getPublicUrls:
  *      Params: (type | string ) eg. 'scripts', 'stylesheets'
  *      Description:
- *        returns array of urls for a type (script, stylesheet, nunjucks etc). do we need to expose this?
+ *        returns array of urls for a type (script, stylesheet, nunjucks etc).
  *    getFileSystemPaths:
  *      Params: (type | string ) eg. 'scripts', 'stylesheets'
  *      Description:
- *        returns array paths to the file in the filesystem for a type (script, stylesheet, nunjucks etc) do we need to expose this?
+ *        returns array paths to the file in the filesystem for a type (script, stylesheet, nunjucks etc)
  *    getPublicUrlAndFileSystemPaths:
  *      Params: (type | string ) eg. 'scripts', 'stylesheets'
  *      Description:
  *        returns Array of objects, each object is an extension and each obj has the filesystem & public url for the given type
  *    getAppConfig:
  *      Params: (type | string ) eg. 'scripts', 'stylesheets'
  *      Description:
- *        to do
+ *        Returns an object containing two keys(scripts & stylesheets), each item contains an array of full paths to specific files.
+ *        This is used in the views to output links and scripts each file.
  *    getAppViews:
- *      Params: (type | string ) eg. 'scripts', 'stylesheets'
+ *      Params: (additionalViews | Array ) eg.extensions.getAppViews([path.join(__dirname, '/app/views/'),path.join(__dirname, '/lib/')])
  *      Description:
- *        To do
+ *        Returns an array of paths to nunjucks templates which is used to configure nunjucks in server.js
  *    setExtensionsByType
  *      Params: N/A
  *      Description: only used for test purposes to reset mocked extensions items to ensure they are up-to-date when the tests run