chore: add lint cache (#2052)

* chore: add lint cache * chore: update * chore: revert dist
vuejs · Feb 8, 2023 · be11db6 · be11db6
1 parent 4fb9dec
commit be11db6
Show file tree

Hide file tree

Showing 66 changed files with 132 additions and 196 deletions.
diff --git a/.eslintignore b/.eslintignore
@@ -1 +1 @@
-dist
+**/dist/**
diff --git a/.gitignore b/.gitignore
@@ -1,5 +1,6 @@
 .DS_Store
 node_modules/
+.eslintcache
 
 # Editor files
 /.idea

diff --git a/.prettierrc.json b/.prettierrc.json
@@ -1,4 +1,6 @@
 {
   "semi": false,
-  "singleQuote": true
+  "singleQuote": true,
+  "trailingComma": "none",
+  "arrowParens": "avoid"
 }
diff --git a/docs/api/options.md b/docs/api/options.md
@@ -169,7 +169,7 @@ You can also pass a function that takes the props as an argument:
 ```js
 shallowMount(Component, {
   scopedSlots: {
-    foo: function(props) {
+    foo: function (props) {
       return this.$createElement('div', props.index)
     }
   }

diff --git a/docs/fr/api/options.md b/docs/fr/api/options.md
@@ -168,7 +168,7 @@ Vous pouvez également passer une fonction qui prend les props comme argument :
 ```js
 shallowMount(Component, {
   scopedSlots: {
-    foo: function(props) {
+    foo: function (props) {
       return this.$createElement('div', props.index)
     }
   }

diff --git a/docs/fr/guides/common-tips.md b/docs/fr/guides/common-tips.md
@@ -221,7 +221,7 @@ En pratique, bien que nous appelions et attendions `setData` pour assurer la mis
 
 ```js
 const transitionStub = () => ({
-  render: function(h) {
+  render: function (h) {
     return this.$options._renderChildren
   }
 })

diff --git a/docs/fr/guides/dom-events.md b/docs/fr/guides/dom-events.md
@@ -144,7 +144,7 @@ Ce composant permet d'incrémenter/décrémenter la quantité à l'aide de diff
     },
 
     watch: {
-      quantity: function(newValue) {
+      quantity: function (newValue) {
         this.$emit('input', newValue)
       }
     }

diff --git a/docs/fr/installation/using-other-test-runners.md b/docs/fr/installation/using-other-test-runners.md
@@ -12,7 +12,7 @@ Vous trouverez ci-dessous une configuration de base de Karma pour Vue Test Utils
 // karma.conf.js
 var webpackConfig = require('./webpack.config.js')
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     frameworks: ['mocha'],
     files: ['test/**/*.spec.js'],

diff --git a/docs/guides/common-tips.md b/docs/guides/common-tips.md
@@ -223,7 +223,7 @@ In practice, although we are calling and awaiting `setData` to ensure the DOM is
 
 ```js
 const transitionStub = () => ({
-  render: function(h) {
+  render: function (h) {
     return this.$options._renderChildren
   }
 })

diff --git a/docs/guides/dom-events.md b/docs/guides/dom-events.md
@@ -142,7 +142,7 @@ This component allows to increment/decrement the quantity using various keys.
     },
 
     watch: {
-      quantity: function(newValue) {
+      quantity: function (newValue) {
         this.$emit('input', newValue)
       }
     }

diff --git a/docs/installation/using-other-test-runners.md b/docs/installation/using-other-test-runners.md
@@ -12,7 +12,7 @@ Following is a basic Karma config for Vue Test Utils:
 // karma.conf.js
 var webpackConfig = require('./webpack.config.js')
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     frameworks: ['mocha'],
     files: ['test/**/*.spec.js'],

diff --git a/docs/ja/api/options.md b/docs/ja/api/options.md
@@ -94,7 +94,7 @@ props を引数に取る関数を渡すことができます。
 ```js
 shallowMount(Component, {
   scopedSlots: {
-    foo: function(props) {
+    foo: function (props) {
       return this.$createElement('div', props.index)
     }
   }

diff --git a/docs/ja/guides/dom-events.md b/docs/ja/guides/dom-events.md
@@ -134,7 +134,7 @@ it('Click on yes button calls our method with argument "yes"', async () => {
       }
     },
     watch: {
-      quantity: function(newValue) {
+      quantity: function (newValue) {
         this.$emit('input', newValue)
       }
     }

diff --git a/docs/ja/guides/getting-started.md b/docs/ja/guides/getting-started.md
@@ -131,7 +131,7 @@ it('will catch the error using done', done => {
 })
 
 it('will catch the error using a promise', () => {
-  return Vue.nextTick().then(function() {
+  return Vue.nextTick().then(function () {
     expect(true).toBe(false)
   })
 })

diff --git a/docs/ja/installation/testing-single-file-components-with-karma.md b/docs/ja/installation/testing-single-file-components-with-karma.md
@@ -36,7 +36,7 @@ npm install --save-dev @vue/test-utils karma karma-chrome-launcher karma-mocha k
 
 var webpackConfig = require('./webpack.config.js')
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     frameworks: ['mocha'],
 
@@ -160,7 +160,7 @@ npm install --save-dev babel-plugin-istanbul
 ```js
 // karma.conf.js
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // ...
 

diff --git a/docs/ru/api/options.md b/docs/ru/api/options.md
@@ -107,7 +107,7 @@ shallowMount(Component, {
 ```js
 shallowMount(Component, {
   scopedSlots: {
-    foo: function(props) {
+    foo: function (props) {
       return this.$createElement('div', props.index)
     }
   }

diff --git a/docs/ru/guides/dom-events.md b/docs/ru/guides/dom-events.md
@@ -136,7 +136,7 @@ describe('Click event', () => {
     },
 
     watch: {
-      quantity: function(newValue) {
+      quantity: function (newValue) {
         this.$emit('input', newValue)
       }
     }

diff --git a/docs/ru/guides/getting-started.md b/docs/ru/guides/getting-started.md
@@ -131,7 +131,7 @@ it('должен отлавливать ошибку с использовани
 })
 
 it('должен отлавливать ошибку с использованием promise', () => {
-  return Vue.nextTick().then(function() {
+  return Vue.nextTick().then(function () {
     expect(true).toBe(false)
   })
 })

diff --git a/docs/ru/installation/testing-single-file-components-with-karma.md b/docs/ru/installation/testing-single-file-components-with-karma.md
@@ -36,7 +36,7 @@ npm install --save-dev @vue/test-utils karma karma-chrome-launcher karma-mocha k
 
 var webpackConfig = require('./webpack.config.js')
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     frameworks: ['mocha'],
 
@@ -160,7 +160,7 @@ npm install --save-dev babel-plugin-istanbul
 ```js
 // karma.conf.js
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // ...
 

diff --git a/docs/zh/api/options.md b/docs/zh/api/options.md
@@ -167,7 +167,7 @@ shallowMount(Component, {
 ```js
 shallowMount(Component, {
   scopedSlots: {
-    foo: function(props) {
+    foo: function (props) {
       return this.$createElement('div', props.index)
     }
   }

diff --git a/docs/zh/guides/common-tips.md b/docs/zh/guides/common-tips.md
@@ -222,7 +222,7 @@ test('should render Foo, then hide it', async () => {
 
 ```js
 const transitionStub = () => ({
-  render: function(h) {
+  render: function (h) {
     return this.$options._renderChildren
   }
 })

diff --git a/docs/zh/guides/dom-events.md b/docs/zh/guides/dom-events.md
@@ -142,7 +142,7 @@ it('Click on yes button calls our method with argument "yes"', async () => {
     },
 
     watch: {
-      quantity: function(newValue) {
+      quantity: function (newValue) {
         this.$emit('input', newValue)
       }
     }

diff --git a/docs/zh/guides/getting-started.md b/docs/zh/guides/getting-started.md
@@ -164,7 +164,7 @@ it('will catch the error using done', done => {
 })
 
 it('will catch the error using a promise', () => {
-  return Vue.nextTick().then(function() {
+  return Vue.nextTick().then(function () {
     expect(true).toBe(false)
   })
 })

diff --git a/docs/zh/installation/testing-single-file-components-with-karma.md b/docs/zh/installation/testing-single-file-components-with-karma.md
@@ -36,7 +36,7 @@ npm install --save-dev @vue/test-utils karma karma-chrome-launcher karma-mocha k
 
 var webpackConfig = require('./webpack.config.js')
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     frameworks: ['mocha'],
 
@@ -160,7 +160,7 @@ npm install --save-dev babel-plugin-istanbul
 ```js
 // karma.conf.js
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // ...
 

diff --git a/package.json b/package.json
@@ -17,10 +17,10 @@
     "docs": "vuepress dev docs",
     "docs:build": "vuepress build docs",
     "flow": "flow check",
-    "lint": "eslint --ext js,vue .",
+    "lint": "eslint --ext js,vue . --cache",
     "lint:docs": "eslint --ext js,vue,md docs --ignore-path .gitignore",
     "lint:fix": "yarn lint -- --fix",
-    "format": "prettier --write \"**/*.{js,json,vue,md}\"",
+    "format": "prettier --write \"**/*.{js,json,vue,md}\" --cache",
     "format:check": "prettier --check \"**/*.{js,json,vue,md}\"",
     "release": "yarn build && yarn test:unit && lerna publish --conventional-commits -m \"chore(release): publish %s\"",
     "test": "yarn format:check && yarn lint && yarn lint:docs && yarn flow && yarn test:types && yarn test:unit -w 1 && yarn test:unit:browser",
@@ -91,7 +91,7 @@
     "karma-spec-reporter": "^0.0.32",
     "karma-webpack": "^4.0.2",
     "lint-staged": "^9.5.0",
-    "prettier": "^1.16.0",
+    "prettier": "^2.8.1",
     "puppeteer": "^5.2.1",
     "rollup-plugin-delete": "^2.0.0",
     "@rollup/plugin-replace": "^2.3.3",

diff --git a/packages/create-instance/create-instance.js b/packages/create-instance/create-instance.js
@@ -107,7 +107,7 @@ export default function createInstance(
   const parentComponentOptions = options.parentComponent || {}
 
   const originalParentComponentProvide = parentComponentOptions.provide
-  parentComponentOptions.provide = function() {
+  parentComponentOptions.provide = function () {
     return {
       ...getValuesFromCallableOption.call(this, originalParentComponentProvide),
       // $FlowIgnore
@@ -116,7 +116,7 @@ export default function createInstance(
   }
 
   const originalParentComponentData = parentComponentOptions.data
-  parentComponentOptions.data = function() {
+  parentComponentOptions.data = function () {
     return {
       ...getValuesFromCallableOption.call(this, originalParentComponentData),
       vueTestUtils_childProps: { ...options.propsData }
@@ -126,7 +126,7 @@ export default function createInstance(
   parentComponentOptions.$_doNotStubChildren = true
   parentComponentOptions.$_isWrapperParent = true
   parentComponentOptions._isFunctionalContainer = componentOptions.functional
-  parentComponentOptions.render = function(h) {
+  parentComponentOptions.render = function (h) {
     return h(
       Constructor,
       createContext(options, scopedSlots, this.vueTestUtils_childProps),

diff --git a/packages/create-instance/create-scoped-slots.js b/packages/create-instance/create-scoped-slots.js
@@ -8,9 +8,9 @@ function isDestructuringSlotScope(slotScope: string): boolean {
   return /^{.*}$/.test(slotScope)
 }
 
-function getVueTemplateCompilerHelpers(
-  _Vue: Component
-): { [name: string]: Function } {
+function getVueTemplateCompilerHelpers(_Vue: Component): {
+  [name: string]: Function
+} {
   // $FlowIgnore
   const vue = new _Vue()
   const helpers = {}
@@ -111,7 +111,7 @@ export default function createScopedSlots(
 
     const slotScope = scopedSlotMatches.match && scopedSlotMatches.match[1]
 
-    scopedSlots[scopedSlotName] = function(props) {
+    scopedSlots[scopedSlotName] = function (props) {
       let res
       if (isFn) {
         res = renderFn.call({ ...helpers }, props)

diff --git a/packages/create-instance/log-events.js b/packages/create-instance/log-events.js
@@ -15,7 +15,7 @@ export function logEvents(
 
 export function addEventLogger(_Vue: Component): void {
   _Vue.mixin({
-    beforeCreate: function() {
+    beforeCreate: function () {
       this.__emitted = Object.create(null)
       this.__emittedByOrder = []
       logEvents(this, this.__emitted, this.__emittedByOrder)

diff --git a/packages/shared/validators.js b/packages/shared/validators.js
@@ -137,10 +137,10 @@ function makeMap(str: string, expectsLowerCase?: boolean) {
     map[list[i]] = true
   }
   return expectsLowerCase
-    ? function(val: string) {
+    ? function (val: string) {
         return map[val.toLowerCase()]
       }
-    : function(val: string) {
+    : function (val: string) {
         return map[val]
       }
 }

diff --git a/packages/test-utils/src/wrapper-array.js b/packages/test-utils/src/wrapper-array.js
@@ -5,7 +5,7 @@ import type VueWrapper from './vue-wrapper'
 import { throwError } from 'shared/util'
 
 export default class WrapperArray implements BaseWrapper {
-  +wrappers: Array<Wrapper | VueWrapper>
+  +wrappers: Array<Wrapper | VueWrapper>;
   +length: number
   selector: Selector | void
 

diff --git a/packages/test-utils/src/wrapper.js b/packages/test-utils/src/wrapper.js
@@ -29,11 +29,11 @@ import createDOMEvent from './create-dom-event'
 import { throwIfInstancesThrew } from './error'
 
 export default class Wrapper implements BaseWrapper {
-  +vnode: VNode | null
+  +vnode: VNode | null;
   +vm: Component | void
   _emitted: { [name: string]: Array<Array<any>> }
-  _emittedByOrder: Array<{ name: string, args: Array<any> }>
-  +element: Element
+  _emittedByOrder: Array<{ name: string, args: Array<any> }>;
+  +element: Element;
   +options: WrapperOptions
   isFunctionalComponent: boolean
   rootNode: VNode | Element
@@ -502,9 +502,7 @@ export default class Wrapper implements BaseWrapper {
         ? value.map((calledWith, index) => {
             const callParams = calledWith.map(param =>
               typeof param === 'object'
-                ? JSON.stringify(param)
-                    .replace(/"/g, '')
-                    .replace(/,/g, ', ')
+                ? JSON.stringify(param).replace(/"/g, '').replace(/,/g, ', ')
                 : param
             )
 

diff --git a/test/resources/components/component-with-computed.vue b/test/resources/components/component-with-computed.vue
@@ -11,11 +11,8 @@ export default {
     message: 'egassem'
   }),
   computed: {
-    reversedMessage: function() {
-      return this.message
-        .split('')
-        .reverse()
-        .join('')
+    reversedMessage: function () {
+      return this.message.split('').reverse().join('')
     }
   }
 }

diff --git a/test/resources/components/component-with-transitions.vue b/test/resources/components/component-with-transitions.vue
@@ -1,9 +1,7 @@
 <template>
   <div>
     <transition>
-      <span v-if="!expanded" data-testid="expanded">
-        Content
-      </span>
+      <span v-if="!expanded" data-testid="expanded"> Content </span>
     </transition>
   </div>
 </template>

diff --git a/test/resources/components/component-with-watch-immediate.vue b/test/resources/components/component-with-watch-immediate.vue
@@ -6,7 +6,7 @@
 
 <script>
 export default {
-  data: function() {
+  data: function () {
     return {
       data1: null
     }