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

fix(types): should unwrap tuple correctly #3820

Merged
merged 3 commits into from Oct 26, 2022

Conversation

hyf0
Copy link
Contributor

@hyf0 hyf0 commented May 23, 2021

fix #3819.

@HcySunYang HcySunYang changed the title fix(reactivity): UnwrapRefSimple<T> should not unwrap tuple fix(types): should unwrap tuple correctly May 23, 2021
@HcySunYang HcySunYang requested a review from pikax May 23, 2021
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch 3 times, most recently from 7b58d85 to a7b5da2 Compare May 23, 2021
antfu
antfu approved these changes May 27, 2021
pikax
pikax approved these changes May 27, 2021
packages/reactivity/src/ref.ts Outdated Show resolved Hide resolved
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch from a7b5da2 to ce0aabe Compare May 27, 2021
@antfu antfu added the ready to merge The PR is ready to be merged. label Aug 11, 2021
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch 4 times, most recently from 09e24ca to b041a8a Compare Sep 12, 2021
@hyf0 hyf0 force-pushed the fix_should_not_unwrap_tuple branch from b041a8a to c77a050 Compare Oct 22, 2021
@yyx990803 yyx990803 merged commit e816812 into vuejs:main Oct 26, 2022
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: types ready to merge The PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unexpected unwrapping for tuple
6 participants