[internal] Lint typescript, json, new eslint rules

.eslintignore

@@ -1,6 +1,28 @@
-__snapshots__
+# unignore hidden files
+!.*
+
+**/__snapshots__
+**/build
+**/cypress/fixtures
+**/dist
+**/dist-test
+**/node_modules
+**/support/fixtures
+**/test/fixtures
+**/vendor
+
+# cli/types is linted by tslint/dtslint
+cli/types
+# packages/example is not linted (think about changing this)
+packages/example
+
+packages/extension/test/helpers/background.js
 packages/server/lib/scaffold/plugins/index.js
 packages/server/lib/scaffold/support/index.js
 packages/server/lib/scaffold/support/commands.js
-packages/server/test/fixtures
-packages/example/cypress
+
+packages/launcher/lib/**/*.js
+
+**/package-lock.json
+**/tsconfig.json
+**/.vscode

.eslintrc

@@ -1,72 +1,9 @@
 {
+  "plugins": [
+    "@cypress/dev"
+  ],
   "extends": [
-    "plugin:cypress-dev/general"
+    "plugin:@cypress/dev/general"
   ],
-  "rules": {
-    "no-multiple-empty-lines": ["error", { "max": 1 } ],
-    "no-else-return": [ "error", { "allowElseIf": false } ],
-    "brace-style": ["error", "1tbs", { "allowSingleLine": false }],
-    "no-unneeded-ternary": ["error"],
-    "array-bracket-newline": ["error", "consistent"],
-    "arrow-body-style": ["error", "always"],
-    "padding-line-between-statements": [
-      "error",
-      {
-        "blankLine": "always",
-        "prev": "*",
-        "next": "return"
-      },
-      {
-        "blankLine": "always",
-        "prev": [
-          "const",
-          "let",
-          "var",
-          "if",
-          "while",
-          "export",
-          "cjs-export",
-          "import",
-          "cjs-import"
-        ],
-        "next": "*"
-      },
-      {
-        "blankLine": "any",
-        "prev": [
-          "const",
-          "let",
-          "var",
-          "import",
-          "cjs-import"
-        ],
-        "next": [
-          "const",
-          "let",
-          "var",
-          "import",
-          "cjs-import"
-        ]
-      }
-    ]
-  },
-  "env": {
-    "es6": true,
-    "node": true
-  },
-  "parserOptions": {
-    "ecmaFeatures": {
-      "legacyDecorators": true
-    }
-  },
-  "overrides": [
-    {
-      "files": [
-        "**/*.jsx"
-      ],
-      "rules": {
-        "arrow-body-style": "off"
-      }
-    }
-  ]
+  "rules": {}
 }

.gitignore

@@ -13,7 +13,6 @@ _test-output
 cypress.zip
 tmp/
 .nyc_output
-.vscode/settings.json
 
 # from extension
 Cached Theme.pak

.vscode/launch.json

@@ -43,7 +43,10 @@
   "compounds": [
     {
       "name": "dev-debug w/ electron",
-      "configurations": ["dev-debug", "electron"]
+      "configurations": [
+        "dev-debug",
+        "electron"
+      ]
     }
   ]
 }

.vscode/settings.json

@@ -0,0 +1,28 @@
+{
+  "eslint.validate": [
+    { 
+      "language": "javascript",
+      "autoFix": true
+    },
+    {
+      "language": "javascriptreact",
+      "autoFix": true
+    },
+    {
+      "language": "typescript",
+      "autoFix": true
+    },
+    {
+      "language": "typescriptreact",
+      "autoFix": true
+    },
+    {
+      "language": "json",
+      "autoFix": true
+    },
+    {
+      "language": "coffeescript",
+      "autoFix": false
+    },
+  ],
+}

.vscode/tasks.json

@@ -1,6 +1,4 @@
 {
-  // See https://go.microsoft.com/fwlink/?LinkId=733558
-  // for the documentation about the tasks.json format
   "version": "2.0.0",
   "tasks": [
     {

.yarnrc

@@ -0,0 +1 @@
+--ignore-engines true

CONTRIBUTING.md

@@ -264,15 +264,16 @@ DEBUG=cypress:launcher
 ### Coding Style
 
 We use [eslint](https://eslint.org/) to lint all JavaScript code and follow rules specified in
-[eslint-plugin-cypress-dev](https://github.com/cypress-io/eslint-plugin-cypress-dev) plugin.
+[@cypress/eslint-plugin-dev](https://github.com/cypress-io/cypress-eslint-plugin-dev) plugin.
 
-When you edit files, you can quickly fix all changed files before committing using
+When you edit files, you can quickly fix all changed files before you commit using
 
 ```bash
-npm run lint-changed
+npm run lint-changed-fix
 ```
 
-When committing files, we run Git pre-commit hook to fix the staged JS files. See the `precommit-lint` script in [package.json](package.json). This might change JS files and you would need to commit the changes again.
+When committing files, we run a Git pre-commit hook to lint the staged JS files. See the `lint-staged` script in [package.json](package.json).  
+If this command fails, you may need to run `npm run lint-changed-fix` and commit those changes.
 
 ### Tests
 

circle.yml

@@ -205,8 +205,8 @@ jobs:
     steps:
       - attach_workspace:
           at: ~/
-      - run: npm run lint
-      - run: npm run all lint
+      ## this will catch .only's in js/coffee as well
+      - run: npm run lint-all
 
   unit-tests:
     <<: *defaults
@@ -237,7 +237,7 @@ jobs:
       - store_test_results:
           path: /tmp/cypress
 
-  lint-typescript:
+  lint-types:
     <<: *defaults
     parallelism: 1
     steps:
@@ -717,7 +717,7 @@ linux-workflow: &linux-workflow
         name: Linux lint
         requires:
           - build
-    - lint-typescript:
+    - lint-types:
         requires:
           - build
     # unit, integration and e2e tests

cli/__snapshots__/build_spec.js

@@ -14,19 +14,19 @@ exports['package.json build outputs expected properties 1'] = {
     "url": "https://github.com/cypress-io/cypress.git"
   },
   "keywords": [
+    "automation",
     "browser",
     "cypress",
     "cypress.io",
-    "automation",
-    "end-to-end",
     "e2e",
+    "end-to-end",
     "integration",
     "mocks",
-    "test",
-    "testing",
     "runner",
     "spies",
-    "stubs"
+    "stubs",
+    "test",
+    "testing"
   ],
   "types": "types",
   "scripts": {

cli/lib/cli.js

@@ -40,6 +40,7 @@ const spaceDelimitedSpecsMsg = (files) => {
     If you are trying to pass multiple spec filenames, separate them by commas instead:
       cypress run --spec spec1,spec2,spec3
   `)
+
   logger.log()
 }
 
@@ -73,8 +74,7 @@ const parseOpts = (opts) => {
     'project', 'spec', 'reporter', 'reporterOptions', 'path', 'destination',
     'port', 'env', 'cypressVersion', 'config', 'record', 'key',
     'browser', 'detached', 'headed', 'global', 'dev', 'force', 'exit',
-    'cachePath', 'cacheList', 'cacheClear', 'parallel', 'group', 'ciBuildId'
-  )
+    'cachePath', 'cacheList', 'cacheClear', 'parallel', 'group', 'ciBuildId')
 
   if (opts.exit) {
     opts = _.omit(opts, 'exit')

cli/lib/tasks/download.js

@@ -157,9 +157,11 @@ const downloadFromUrl = ({ url, downloadDestination, progress }) => {
   })
 }
 
-const start = ({ version, downloadDestination, progress }) => {
+const start = (opts) => {
+  let { version, downloadDestination, progress } = opts
+
   if (!downloadDestination) {
-    la(is.unemptyString(downloadDestination), 'missing download dir', arguments)
+    la(is.unemptyString(downloadDestination), 'missing download dir', opts)
   }
 
   if (!progress) {

cli/lib/tasks/install.js

@@ -161,7 +161,9 @@ const start = (options = {}) => {
       debug('environment variable CYPRESS_INSTALL_BINARY = 0, skipping install')
       logger.log(
         stripIndent`
-        ${chalk.yellow('Note:')} Skipping binary installation: Environment variable CYPRESS_INSTALL_BINARY = 0.`)
+        ${chalk.yellow('Note:')} Skipping binary installation: Environment variable CYPRESS_INSTALL_BINARY = 0.`
+      )
+
       logger.log()
 
       return Promise.resolve()
@@ -184,7 +186,9 @@ const start = (options = {}) => {
         ${chalk.yellow('Note:')} Overriding Cypress cache directory to: ${chalk.cyan(envCache)}
 
               Previous installs of Cypress may not be found.
-      `)
+      `
+    )
+
     logger.log()
   }
 
@@ -217,6 +221,7 @@ const start = (options = {}) => {
     logger.log(stripIndent`
       Cypress ${chalk.green(binaryVersion)} is installed in ${chalk.cyan(installDir)}
       `)
+
     logger.log()
 
     if (options.force) {
@@ -254,6 +259,7 @@ const start = (options = {}) => {
             These versions may not work properly together.
         `)
       )
+
       logger.log()
     }
 

cli/lib/tasks/state.js

@@ -118,7 +118,8 @@ const writeBinaryVerifiedAsync = (verified, binaryDir) => {
     return fs.outputJsonAsync(
       getBinaryStatePath(binaryDir),
       _.extend(contents, { verified }),
-      { spaces: 2 })
+      { spaces: 2 }
+    )
   })
 }
 

cli/lib/tasks/verify.js

@@ -260,6 +260,7 @@ const start = (options = {}) => {
 
       This overrides the default Cypress binary path used.
     `)
+
     logger.log()
 
     return util.isExecutableAsync(envBinaryPath)

cli/lib/util.js

@@ -71,6 +71,7 @@ const logBrokenGtkDisplayWarning = () => {
 
     Cypress will attempt to fix the problem and rerun.
   `)
+
   logger.warn()
 }