Skip to content

Commit

Permalink
fix: remove compilerOptions from config
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 7, 2022
1 parent 99b4cae commit 3bf21e4
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
Original file line number Diff line number Diff line change
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
Original file line number Diff line number Diff line change
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 3bf21e4

Please sign in to comment.