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(compiler-sfc): rename expose variable name #7949

Merged
merged 1 commit into from
Mar 28, 2023
Merged

fix(compiler-sfc): rename expose variable name #7949

merged 1 commit into from
Mar 28, 2023

Conversation

sxzz
Copy link
Member

@sxzz sxzz commented Mar 23, 2023

rename expose to __expose to avoid duplicate identifier

relate #7890

@sxzz
Copy link
Member Author

sxzz commented Mar 23, 2023

/ecosystem-ci run

@vue-bot
Copy link
Contributor

vue-bot commented Mar 23, 2023

📝 Ran ecosystem CI: Open

suite result
naive-ui ❌ failure
nuxt ❌ failure
pinia ✅ success
router ✅ success
test-utils ❌ failure
vant ✅ success
vite-plugin-vue ✅ success
vitepress ✅ success
vue-macros ❌ failure
vuetify ✅ success
vueuse ✅ success

rename `expose` to `__expose` to avoid duplicate identifier

relate #7890
@yyx990803 yyx990803 merged commit c839129 into vuejs:main Mar 28, 2023
@sxzz sxzz deleted the fix/expose-name branch March 28, 2023 09:51
@yyx990803
Copy link
Member

yyx990803 commented Mar 29, 2023

I should've looked at ecosystem CI results before merging 😅

IAmSSH pushed a commit to IAmSSH/core that referenced this pull request May 14, 2023
…xpose` (vuejs#7949)

rename `expose` to `__expose` to avoid duplicate identifier

relate vuejs#7890
IAmSSH pushed a commit to IAmSSH/core that referenced this pull request May 14, 2023
@sxzz sxzz mentioned this pull request Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants