Skip to content

Commit

Permalink
fix: remove compilerOptions from config (#1408)
Browse files Browse the repository at this point in the history
This option was not used, and not exposed in the typings.
This was misleading as developers can imagine that setting `compilerOptions` in the config is the proper way to configure Vue.
This should be done either by:
- setting the options via https://github.com/vuejs/vue-jest#global-jest-options for Jest and vue-jest
- setting the options in Vite directly for vitest
  • Loading branch information
cexbrayat committed Apr 8, 2022
1 parent 91b4603 commit 7fedff7
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
4 changes: 0 additions & 4 deletions src/utils.ts
Expand Up @@ -26,10 +26,6 @@ function mergeAppConfig(
globalProperties: {
...configGlobalConfig?.globalProperties,
...mountGlobalConfig?.globalProperties
},
compilerOptions: {
...configGlobalConfig?.compilerOptions,
...mountGlobalConfig?.compilerOptions
}
}
}
Expand Down
6 changes: 1 addition & 5 deletions tests/features/compat.spec.ts
Expand Up @@ -3,13 +3,9 @@ import { mount } from '../../src'

jest.mock('vue', () => mockVue)

const { configureCompat, extend, defineComponent, h, config } = mockVue as any
const { configureCompat, extend, defineComponent, h } = mockVue as any

describe('@vue/compat build', () => {
beforeAll(() => {
config.compilerOptions.whitespace = 'condense'
})

describe.each(['suppress-warning', false])(
'when RENDER_FUNCTION compat is %p',
(RENDER_FUNCTION) => {
Expand Down

0 comments on commit 7fedff7

Please sign in to comment.