Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: use helper function for merging vitest config #8985

Merged
merged 1 commit into from
Oct 26, 2023
Merged

chore: use helper function for merging vitest config #8985

merged 1 commit into from
Oct 26, 2023

Conversation

Alfred-Skyblue
Copy link
Member

Merge the vitest configurations using the mergeConfig function.

@sxzz sxzz added the ready to merge The PR is ready to be merged. label Aug 21, 2023
@sxzz sxzz force-pushed the test-vitest-config branch from 22826d2 to 4798143 Compare August 21, 2023 07:44
@github-actions
Copy link

Size Report

Bundles

File Size Gzip Brotli
runtime-dom.global.prod.js 85.8 kB 32.6 kB 29.4 kB
vue.global.prod.js 132 kB 49.3 kB 44.3 kB

Usages

Name Size Gzip Brotli
createApp 47.7 kB 18.9 kB 17.2 kB
createSSRApp 50.5 kB 19.9 kB 18.2 kB
defineCustomElement 50.1 kB 19.6 kB 17.9 kB
overall 61.1 kB 23.7 kB 21.6 kB

@sxzz sxzz changed the title chore(vitest): use the mergeConfig function to merge configurations chore: use helper function for merging vitest config Oct 20, 2023
@haoqunjiang haoqunjiang merged commit 6d7360d into vuejs:main Oct 26, 2023
@Alfred-Skyblue Alfred-Skyblue deleted the test-vitest-config branch October 26, 2023 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sxzz sxzz sxzz approved these changes

@edison1105 edison1105 edison1105 approved these changes

@haoqunjiang haoqunjiang haoqunjiang approved these changes

Assignees
No one assigned
Labels
ready to merge The PR is ready to be merged.
Projects
Issue / PR Management
Status: Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Alfred-Skyblue @haoqunjiang @edison1105 @sxzz