style(addons): use oclif builtin log()

netlify · Nov 3, 2018 · 035a93c · 035a93c
1 parent 5b19fc5
commit 035a93c
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 30 deletions.
diff --git a/src/commands/addons/create.js b/src/commands/addons/create.js
@@ -11,24 +11,24 @@ class addonsCreateCommand extends Command {
     const addonName = args.name
 
     if (!addonName) {
-      console.log('Please provide an addon name to provision')
-      // console.log(util.inspect(myObject, false, null, true /* enable colors */))
+      this.log('Please provide an addon name to provision')
+      // this.log(util.inspect(myObject, false, null, true /* enable colors */))
       this.exit()
     }
 
     const siteId = site.id
 
     if (!siteId) {
-      console.log('No site id found, please run inside a site folder or `netlify link`')
+      this.log('No site id found, please run inside a site folder or `netlify link`')
       return false
     }
 
     const siteData = await api.getSite({ siteId })
-    // console.log(site)
+    // this.log(site)
     const addons = await getAddons(siteId, accessToken)
 
     if (typeof addons === 'object' && addons.error) {
-      console.log('API Error', addons)
+      this.log('API Error', addons)
       return false
     }
 
@@ -45,9 +45,9 @@ class addonsCreateCommand extends Command {
     const rawFlags = parseRawFlags(raw)
 
     if (currentAddon.id) {
-      console.log(`Addon ${addonName} already exists for ${siteData.name}`)
-      console.log(`> Run \`netlify addons:update ${addonName}\` to update settings`)
-      console.log(`> Run \`netlify addons:delete ${addonName}\` to delete this addon`)
+      this.log(`Addon ${addonName} already exists for ${siteData.name}`)
+      this.log(`> Run \`netlify addons:update ${addonName}\` to update settings`)
+      this.log(`> Run \`netlify addons:delete ${addonName}\` to delete this addon`)
       return false
     }
 
@@ -59,11 +59,11 @@ class addonsCreateCommand extends Command {
     const addonResponse = await createAddon(settings, accessToken)
 
     if (addonResponse.code === 404) {
-      console.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
+      this.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
       return false
     }
     this.log(`Addon "${addonName}" created for ${siteData.name}`)
-    // console.log(addonResponse)
+    // this.log(addonResponse)
   }
 }
 

diff --git a/src/commands/addons/delete.js b/src/commands/addons/delete.js
@@ -13,14 +13,14 @@ class addonsDeleteCommand extends Command {
     const siteId = site.id
 
     if (!siteId) {
-      console.log('No site id found, please run inside a site folder or `netlify link`')
+      this.log('No site id found, please run inside a site folder or `netlify link`')
       return false
     }
 
     const addons = await getAddons(siteId, accessToken)
 
     if (typeof addons === 'object' && addons.error) {
-      console.log('API Error', addons)
+      this.log('API Error', addons)
       return false
     }
 
@@ -34,11 +34,11 @@ class addonsDeleteCommand extends Command {
 
     // If we need flags here
     // const rawFlags = parseRawFlags(raw)
-    // console.log('rawFlags', rawFlags)
+    // this.log('rawFlags', rawFlags)
 
     if (!currentAddon.id) {
-      console.log(`No addon "${addonName}" found for site. Addon already deleted or never existed!`)
-      console.log(`> Run \`netlify addons:create ${addonName}\` to create an instance for this site`)
+      this.log(`No addon "${addonName}" found for site. Addon already deleted or never existed!`)
+      this.log(`> Run \`netlify addons:create ${addonName}\` to create an instance for this site`)
       return false
     }
 
@@ -50,7 +50,7 @@ class addonsDeleteCommand extends Command {
     const addonResponse = await deleteAddon(settings, accessToken)
 
     if (addonResponse.code === 404) {
-      console.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
+      this.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
       return false
     }
     this.log(`Addon "${addonName}" deleted`)

diff --git a/src/commands/addons/list.js b/src/commands/addons/list.js
@@ -11,27 +11,27 @@ class AddonsListCommand extends Command {
     const siteId = site.id
 
     if (!siteId) {
-      console.log('No site id found, please run inside a site folder or `netlify link`')
+      this.log('No site id found, please run inside a site folder or `netlify link`')
       return false
     }
 
     const siteData = await api.getSite({ siteId })
 
     const addons = await getAddons(siteId, accessToken)
     if (!addons || !addons.length) {
-      console.log(`No addons currently installed for ${siteData.name}`)
-      console.log(`> Run \`netlify addons:create addon-namespace\` to install an addon`)
+      this.log(`No addons currently installed for ${siteData.name}`)
+      this.log(`> Run \`netlify addons:create addon-namespace\` to install an addon`)
       return false
     }
 
     // Json response for piping commands
     if (flags.json) {
-      console.log(JSON.stringify(addons, null, 2))
+      this.log(JSON.stringify(addons, null, 2))
       return false
     }
 
     const addonData = addons.map(addon => {
-      // console.log('addon', addon)
+      // this.log('addon', addon)
       return {
         namespace: addon.service_path.replace('/.netlify/', ''),
         name: addon.service_name,
@@ -40,7 +40,7 @@ class AddonsListCommand extends Command {
     })
 
     // Build a table out of addons
-    console.log(`site: ${siteData.name}`)
+    this.log(`site: ${siteData.name}`)
     const table = new AsciiTable(`Currently Installed addons`)
 
     table.setHeading('NameSpace', 'Name', 'Instance Id')
@@ -49,7 +49,7 @@ class AddonsListCommand extends Command {
       table.addRow(s.namespace, s.name, s.id)
     })
     // Log da addons
-    console.log(table.toString())
+    this.log(table.toString())
   }
 }
 

diff --git a/src/commands/addons/update.js b/src/commands/addons/update.js
@@ -13,16 +13,16 @@ class addonsUpdateCommand extends Command {
     const siteId = this.netlify.site.id
 
     if (!siteId) {
-      console.log('No site id found, please run inside a site folder or `netlify link`')
+      this.log('No site id found, please run inside a site folder or `netlify link`')
       return false
     }
 
     const site = await this.netlify.api.getSite({ siteId })
-    // console.log(site)
+    // this.log(site)
     const addons = await getAddons(siteId, accessToken)
 
     if (typeof addons === 'object' && addons.error) {
-      console.log('API Error', addons)
+      this.log('API Error', addons)
       return false
     }
 
@@ -35,8 +35,8 @@ class addonsUpdateCommand extends Command {
     }, addons)
 
     if (!currentAddon || !currentAddon.id) {
-      console.log(`Addon ${addonName} doesnt exist for ${site.name}`)
-      console.log(`> Run \`netlify addons:create ${addonName}\` to create an instance for this site`)
+      this.log(`Addon ${addonName} doesnt exist for ${site.name}`)
+      this.log(`> Run \`netlify addons:create ${addonName}\` to create an instance for this site`)
       return false
     }
 
@@ -59,10 +59,10 @@ class addonsUpdateCommand extends Command {
     }
 
     const updateAddonResponse = await updateAddon(settings, accessToken)
-    // console.log('addonResponse', updateAddonResponse)
+    // this.log('addonResponse', updateAddonResponse)
 
     if (updateAddonResponse.code === 404) {
-      console.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
+      this.log(`No addon "${addonName}" found. Please double check your addon name and try again`)
       return false
     }
     this.log(`Addon "${addonName}" updated`)