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.

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

chore: write a test for renderHelpers #7612

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

thomasrockhu-codecov
Copy link

Just wanted to go through the exercise of writing tests and noticed there weren't any tests on this file (runtime-core/renderHelpers). Let me know if this looks good or if I can do anything to help!

@netlify
Copy link

netlify bot commented Jan 31, 2023

Deploy Preview for vuejs-coverage failed.

Name Link
🔨 Latest commit aadd76c
🔍 Latest deploy log https://app.netlify.com/sites/vuejs-coverage/deploys/63d9560e1d659100087d6f93

@haoqunjiang haoqunjiang added the 🧹 p1-chore Priority 1: this doesn't change code behavior. label Apr 1, 2024
Copy link

github-actions bot commented Apr 1, 2024

Size Report

Bundles

File Size Gzip Brotli
runtime-dom.global.prod.js 90.6 kB 34.4 kB 31 kB
vue.global.prod.js 148 kB 53.7 kB 47.9 kB

Usages

Name Size Gzip Brotli
createApp 50.8 kB 19.8 kB 18.1 kB
createSSRApp 54.1 kB 21.2 kB 19.3 kB
defineCustomElement 53.1 kB 20.6 kB 18.8 kB
overall 64.4 kB 24.9 kB 22.6 kB

@edison1105 edison1105 added the ready to merge The PR is ready to be merged. label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 p1-chore Priority 1: this doesn't change code behavior. ready to merge The PR is ready to be merged.
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

5 participants