Fix: Fixing prettier issues, and add to scripts test:prettier

netlify · Sep 17, 2019 · f41e236 · f41e236
1 parent 2bff6e4
commit f41e236
Show file tree

Hide file tree

Showing 157 changed files with 2,195 additions and 2,585 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,8 +1,5 @@
 {
-  "plugins": [
-    "node",
-    "prettier"
-  ],
+  "plugins": ["node", "prettier"],
   "parserOptions": {
     "sourceType": "module",
     "ecmaFeatures": {

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -17,7 +17,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 ### Commits
 
-- RulesProxy: Make sure _redirects works with dev [`27f92f2`](https://github.com/netlify/cli/commit/27f92f23148f7250e63a1586744d155a54751cd5)
+- RulesProxy: Make sure \_redirects works with dev [`27f92f2`](https://github.com/netlify/cli/commit/27f92f23148f7250e63a1586744d155a54751cd5)
 - update logo [`3984bbb`](https://github.com/netlify/cli/commit/3984bbb1c4177fb410f9100d83baac7875ea0025)
 - Dev: Make stdio more maintainable [`4cbdb02`](https://github.com/netlify/cli/commit/4cbdb02ca85c7680b9c469aad5ce725e3975f86f)
 
@@ -29,11 +29,11 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 - Dev: Make functions port configureable [`#525`](https://github.com/netlify/cli/pull/525)
 - Fix port for proxy redirects [`#520`](https://github.com/netlify/cli/pull/520)
 - Rules Proxy: Pass on query parameters [`#519`](https://github.com/netlify/cli/pull/519)
-- Import rules-proxy and  and redirect-parser [`#518`](https://github.com/netlify/cli/pull/518)
+- Import rules-proxy and and redirect-parser [`#518`](https://github.com/netlify/cli/pull/518)
 - Consider sites from different team in Netlify link [`#500`](https://github.com/netlify/cli/pull/500)
 - Improve prompts for `link` command [`#502`](https://github.com/netlify/cli/pull/502)
 - Add Parcel detector [`#505`](https://github.com/netlify/cli/pull/505)
-- rename deploy path  [`#508`](https://github.com/netlify/cli/pull/508)
+- rename deploy path [`#508`](https://github.com/netlify/cli/pull/508)
 - Improve analytics [`#513`](https://github.com/netlify/cli/pull/513)
 - Update ISSUE_TEMPLATE.md [`#504`](https://github.com/netlify/cli/pull/504)
 - Neltify -> Netlify [`#512`](https://github.com/netlify/cli/pull/512)
@@ -383,7 +383,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 - update netlify dev plugin [`514d759`](https://github.com/netlify/cli/commit/514d7591609fbd645c1de87f47200099e6919abe)
 - Add a test for raw parsing [`deb4cb9`](https://github.com/netlify/cli/commit/deb4cb97cf0ef12efba83b07c719064e9e16342e)
-- Use conventional file casing  [`60ad9d0`](https://github.com/netlify/cli/commit/60ad9d07c5a43ea24e78b63d70955e1a11dacc13)
+- Use conventional file casing [`60ad9d0`](https://github.com/netlify/cli/commit/60ad9d07c5a43ea24e78b63d70955e1a11dacc13)
 
 ## [v2.11.7](https://github.com/netlify/cli/compare/v2.11.6...v2.11.7) - 2019-04-10
 
@@ -597,7 +597,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 ### Commits
 
 - Run `npm up` [`29f16fa`](https://github.com/netlify/cli/commit/29f16fa376a55093b09d0c14a67e6ab68a5718f7)
-- Update js-client providing zip-n-ship capabilities  [`f9774a2`](https://github.com/netlify/cli/commit/f9774a29781c37d10875563c6221c2b92d2d7800)
+- Update js-client providing zip-n-ship capabilities [`f9774a2`](https://github.com/netlify/cli/commit/f9774a29781c37d10875563c6221c2b92d2d7800)
 - Update deps [`985e8da`](https://github.com/netlify/cli/commit/985e8da37e162fefabfd27ac5bf6e567ba7ea790)
 
 ## [v2.6.6](https://github.com/netlify/cli/compare/v2.6.5...v2.6.6) - 2019-01-31
@@ -608,7 +608,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 ### Commits
 
-- Run prune before shrink-wrap  [`761fcc5`](https://github.com/netlify/cli/commit/761fcc5660c8ac2a293afde661a129a0c8fbee84)
+- Run prune before shrink-wrap [`761fcc5`](https://github.com/netlify/cli/commit/761fcc5660c8ac2a293afde661a129a0c8fbee84)
 
 ## [v2.6.5](https://github.com/netlify/cli/compare/v2.6.4...v2.6.5) - 2019-01-31
 
@@ -645,7 +645,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 ### Commits
 
-- Update js-client to fix rate limit handling  [`32590eb`](https://github.com/netlify/cli/commit/32590ebf2aa9788733908afee3aefd87f21e9bb8)
+- Update js-client to fix rate limit handling [`32590eb`](https://github.com/netlify/cli/commit/32590ebf2aa9788733908afee3aefd87f21e9bb8)
 
 ## [v2.6.2](https://github.com/netlify/cli/compare/v2.6.1...v2.6.2) - 2019-01-19
 
@@ -735,7 +735,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 ### Commits
 
-- Fix missing dependencies  [`4dd06ad`](https://github.com/netlify/cli/commit/4dd06ad5bf53d62355e2f126bcc641786dba4139)
+- Fix missing dependencies [`4dd06ad`](https://github.com/netlify/cli/commit/4dd06ad5bf53d62355e2f126bcc641786dba4139)
 
 ## [v2.2.2](https://github.com/netlify/cli/compare/v2.2.1...v2.2.2) - 2018-11-26
 
@@ -859,7 +859,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 ### Commits
 
 - add root to this.netlify.site [`9104cf2`](https://github.com/netlify/cli/commit/9104cf2df283e19b4565daeb7f9c7a695a5bc419)
-- Use dirname, not base name  [`73b594c`](https://github.com/netlify/cli/commit/73b594c1bb8dc024ecca8d1a1c9a42ab0b953317)
+- Use dirname, not base name [`73b594c`](https://github.com/netlify/cli/commit/73b594c1bb8dc024ecca8d1a1c9a42ab0b953317)
 
 ## [v2.0.0-beta.8](https://github.com/netlify/cli/compare/v2.0.0-beta.7...v2.0.0-beta.8) - 2018-10-09
 

diff --git a/CODE_OF_CONDUCT.md b/CODE_OF_CONDUCT.md
@@ -14,21 +14,21 @@ orientation.
 Examples of behavior that contributes to creating a positive environment
 include:
 
-* Using welcoming and inclusive language
-* Being respectful of differing viewpoints and experiences
-* Gracefully accepting constructive criticism
-* Focusing on what is best for the community
-* Showing empathy towards other community members
+- Using welcoming and inclusive language
+- Being respectful of differing viewpoints and experiences
+- Gracefully accepting constructive criticism
+- Focusing on what is best for the community
+- Showing empathy towards other community members
 
 Examples of unacceptable behavior by participants include:
 
-* The use of sexualized language or imagery and unwelcome sexual attention or
-advances
-* Trolling, insulting/derogatory comments, and personal or political attacks
-* Public or private harassment
-* Publishing others' private information, such as a physical or electronic
+- The use of sexualized language or imagery and unwelcome sexual attention or
+  advances
+- Trolling, insulting/derogatory comments, and personal or political attacks
+- Public or private harassment
+- Publishing others' private information, such as a physical or electronic
   address, without explicit permission
-* Other conduct which could reasonably be considered inappropriate in a
+- Other conduct which could reasonably be considered inappropriate in a
   professional setting
 
 ## Our Responsibilities

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -66,7 +66,7 @@ We actively welcome your pull requests.
 
 1. Clean out local `node_modules`. `rm -rf node_modules`. This is to counteract any oddities that may arise during the `shrinkwrap` process that happens when cli is released.
 2. Install dependencies. `npm install`.
-3. `npm version [major, minor, patch]`  Generate changelog and bump version.
+3. `npm version [major, minor, patch]` Generate changelog and bump version.
 4. `npm publish` Publish to npm, push version commit + tag, push latest CHANGELOG entry to GitHub release page.
 
 ## License

diff --git a/README.md b/README.md
@@ -1,4 +1,5 @@
 # Netlify CLI
+
 [![npm version][npm-img]][npm] [![build status][azure-img]][azure] [![coverage][coverage-img]][coverage] [![dependencies][david-img]][david] [![downloads][dl-img]][dl] [![netlify-status][netlify-img]][netlify]
 
 Welcome to the Netlify CLI! The new 2.0 version was rebuilt from the ground up to help improve the site building experience.
@@ -15,20 +16,20 @@ Welcome to the Netlify CLI! The new 2.0 version was rebuilt from the ground up t
 - [Usage](#usage)
 - [Documentation](#documentation)
 - [Commands](#commands)
-  * [addons](#addons)
-  * [api](#api)
-  * [deploy](#deploy)
-  * [dev](#dev)
-  * [functions](#functions)
-  * [init](#init)
-  * [link](#link)
-  * [login](#login)
-  * [open](#open)
-  * [sites](#sites)
-  * [status](#status)
-  * [switch](#switch)
-  * [unlink](#unlink)
-  * [watch](#watch)
+  - [addons](#addons)
+  - [api](#api)
+  - [deploy](#deploy)
+  - [dev](#dev)
+  - [functions](#functions)
+  - [init](#init)
+  - [link](#link)
+  - [login](#login)
+  - [open](#open)
+  - [sites](#sites)
+  - [status](#status)
+  - [switch](#switch)
+  - [unlink](#unlink)
+  - [watch](#watch)
 - [Contributing](#contributing)
 - [Development](#development)
 - [License](#license)
@@ -66,18 +67,18 @@ For a full command reference, see the list below, or visit [cli.netlify.com](htt
 ## Commands
 
 <!-- AUTO-GENERATED-CONTENT:START (GENERATE_COMMANDS_LIST) -->
+
 ### [addons](/docs/commands/addons.md)
 
 (Beta) Manage Netlify Add-ons
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`addons:auth`](/docs/commands/addons.md#addonsauth) | Login to add-on provider  |
-| [`addons:config`](/docs/commands/addons.md#addonsconfig) | Configure add-on settings  |
-| [`addons:create`](/docs/commands/addons.md#addonscreate) | Add an add-on extension to your site  |
-| [`addons:delete`](/docs/commands/addons.md#addonsdelete) | Remove an add-on extension to your site  |
-| [`addons:list`](/docs/commands/addons.md#addonslist) | List currently installed add-ons for site  |
-
+| Subcommand                                               | description                               |
+| :------------------------------------------------------- | :---------------------------------------- |
+| [`addons:auth`](/docs/commands/addons.md#addonsauth)     | Login to add-on provider                  |
+| [`addons:config`](/docs/commands/addons.md#addonsconfig) | Configure add-on settings                 |
+| [`addons:create`](/docs/commands/addons.md#addonscreate) | Add an add-on extension to your site      |
+| [`addons:delete`](/docs/commands/addons.md#addonsdelete) | Remove an add-on extension to your site   |
+| [`addons:list`](/docs/commands/addons.md#addonslist)     | List currently installed add-ons for site |
 
 ### [api](/docs/commands/api.md)
 
@@ -91,21 +92,19 @@ Create a new deploy from the contents of a folder
 
 Local dev server
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`dev:exec`](/docs/commands/dev.md#devexec) | Exec command  |
-
+| Subcommand                                  | description  |
+| :------------------------------------------ | :----------- |
+| [`dev:exec`](/docs/commands/dev.md#devexec) | Exec command |
 
 ### [functions](/docs/commands/functions.md)
 
 Manage netlify functions
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`functions:build`](/docs/commands/functions.md#functionsbuild) | build functions locally  |
-| [`functions:create`](/docs/commands/functions.md#functionscreate) | create a new function locally  |
-| [`functions:invoke`](/docs/commands/functions.md#functionsinvoke) | trigger a function while in netlify dev with simulated data, good for testing function calls including Netlify's Event Triggered Functions  |
-
+| Subcommand                                                        | description                                                                                                                                |
+| :---------------------------------------------------------------- | :----------------------------------------------------------------------------------------------------------------------------------------- |
+| [`functions:build`](/docs/commands/functions.md#functionsbuild)   | build functions locally                                                                                                                    |
+| [`functions:create`](/docs/commands/functions.md#functionscreate) | create a new function locally                                                                                                              |
+| [`functions:invoke`](/docs/commands/functions.md#functionsinvoke) | trigger a function while in netlify dev with simulated data, good for testing function calls including Netlify's Event Triggered Functions |
 
 ### [init](/docs/commands/init.md)
 
@@ -123,31 +122,28 @@ Login to your Netlify account
 
 Open settings for the site linked to the current folder
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`open:admin`](/docs/commands/open.md#openadmin) | Opens current site admin UI in Netlify  |
-| [`open:site`](/docs/commands/open.md#opensite) | Opens current site url in browser  |
-
+| Subcommand                                       | description                            |
+| :----------------------------------------------- | :------------------------------------- |
+| [`open:admin`](/docs/commands/open.md#openadmin) | Opens current site admin UI in Netlify |
+| [`open:site`](/docs/commands/open.md#opensite)   | Opens current site url in browser      |
 
 ### [sites](/docs/commands/sites.md)
 
 Handle various site operations
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`sites:create`](/docs/commands/sites.md#sitescreate) | Create an empty site (advanced)  |
-| [`sites:delete`](/docs/commands/sites.md#sitesdelete) | Delete a site  |
-| [`sites:list`](/docs/commands/sites.md#siteslist) | List all sites you have access to  |
-
+| Subcommand                                            | description                       |
+| :---------------------------------------------------- | :-------------------------------- |
+| [`sites:create`](/docs/commands/sites.md#sitescreate) | Create an empty site (advanced)   |
+| [`sites:delete`](/docs/commands/sites.md#sitesdelete) | Delete a site                     |
+| [`sites:list`](/docs/commands/sites.md#siteslist)     | List all sites you have access to |
 
 ### [status](/docs/commands/status.md)
 
 Print status information
 
-| Subcommand | description  |
-|:--------------------------- |:-----|
-| [`status:hooks`](/docs/commands/status.md#statushooks) | Print hook information of the linked site  |
-
+| Subcommand                                             | description                               |
+| :----------------------------------------------------- | :---------------------------------------- |
+| [`status:hooks`](/docs/commands/status.md#statushooks) | Print hook information of the linked site |
 
 ### [switch](/docs/commands/switch.md)
 
@@ -161,7 +157,6 @@ Unlink a local folder from a Netlify site
 
 Watch for site deploy to finish
 
-
 <!-- AUTO-GENERATED-CONTENT:END -->
 
 ## Contributing

diff --git a/azure-pipelines-template.yml b/azure-pipelines-template.yml
@@ -6,18 +6,18 @@ jobs:
     steps:
       - task: NodeTool@0
         inputs:
-          versionSpec: "10.x"
-        displayName: "Install Node.js"
+          versionSpec: '10.x'
+        displayName: 'Install Node.js'
       - script: |
           node --version
           npm --version
-        displayName: "Print node + npm versions"
+        displayName: 'Print node + npm versions'
       - task: Npm@1
         inputs:
           command: install
-        displayName: "Install deps with npm"
+        displayName: 'Install deps with npm'
       - task: Npm@1
         inputs:
           command: custom
           customCommand: test
-        displayName: "Run tests"
+        displayName: 'Run tests'
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -1,15 +1,15 @@
 jobs:
-- template: azure-pipelines-template.yml
-  parameters:
-    name: macOS
-    vmImage: macOS-10.13
+  - template: azure-pipelines-template.yml
+    parameters:
+      name: macOS
+      vmImage: macOS-10.13
 
-- template: azure-pipelines-template.yml
-  parameters:
-    name: Linux
-    vmImage: ubuntu-16.04
+  - template: azure-pipelines-template.yml
+    parameters:
+      name: Linux
+      vmImage: ubuntu-16.04
 
-- template: azure-pipelines-template.yml
-  parameters:
-    name: Windows
-    vmImage: vs2017-win2016
+  - template: azure-pipelines-template.yml
+    parameters:
+      name: Windows
+      vmImage: vs2017-win2016
diff --git a/package.json b/package.json
@@ -47,6 +47,7 @@
     "ava": "ava --verbose",
     "test:ava": "nyc --reporter=lcov ava --verbose && nyc report",
     "test:lint": "eslint src",
+    "test:prettier": "prettier --write --loglevel warn 'src/**/*.js' '*.{js,md,yml,json}'",
     "test:deps": "dependency-check ./package.json --entry \"src/**/!(*.test).js\" --unused --missing --no-dev --no-peer -i @oclif/plugin-not-found -i @oclif/config -i @oclif/plugin-help -i @oclif/plugin-plugins",
     "test:cli-version": "npm run start -- --version",
     "test:cli-help": "npm run start -- --help",

diff --git a/src/commands/addons/auth.js b/src/commands/addons/auth.js
@@ -40,9 +40,9 @@ class AddonsAuthCommand extends Command {
     await this.config.runHook('analytics', {
       eventName: 'command',
       payload: {
-        command: "addons:auth",
-      },
-    });
+        command: 'addons:auth'
+      }
+    })
 
     this.log()
     this.log(`Opening ${addonName} add-on admin URL:`)