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

feature: support preact #67

Merged
merged 1 commit into from
Dec 30, 2023
Merged

feature: support preact #67

merged 1 commit into from
Dec 30, 2023

Conversation

rhliu1996
Copy link
Contributor

@rhliu1996 rhliu1996 commented Dec 23, 2023

Completed 1676 patch-1 rhliu1996 1 Powered by Pull Request Badge

支持preact

🤔 这个变动的性质是?/ What is the nature of this change?

  • 新特性提交 / New feature
  • bug 修复 / Fix bug
  • 样式优化 / Style optimization
  • 代码风格优化 / Code style optimization
  • 性能优化 / Performance optimization
  • 构建优化 / Build optimization
  • 网站、文档、Demo 改进 / Website, documentation, demo improvements
  • 重构代码或样式 / Refactor code or style
  • 测试相关 / Test related
  • 其他 / Other

🔗 相关 Issue / Related Issue

💡 需求背景和解决方案 / Background or solution

📝 更新日志 / Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

支持preact
Copy link

vercel bot commented Dec 23, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @innocces on Vercel.

@innocces first needs to authorize it.

@innocces
Copy link
Owner

可以看下 #47 里的评论. 实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {
  react: 'preact'
}
```.
所以在 preact 模式下. react 依然可以被找到

@innocces innocces added the keep question need code label Dec 24, 2023
@innocces innocces changed the title Update index.ts feature: support preact Dec 24, 2023
@rhliu1996
Copy link
Contributor Author

可以看下 #47 里的评论.实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {
  react: 'preact'
}
```.
所以在 preact 模式下. react 依然可以被找到

并不会我最近工程尝试过

@rhliu1996
Copy link
Contributor Author

可以看下 #47 里的评论.实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {
  react: 'preact'
}
```.
所以在 preact 模式下. react 依然可以被找到

官方cli创建的是没有别名的如果这样直接提交可以减少眼瞎的问题

@innocces
Copy link
Owner

可以看下 #47 里的评论.实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {
  react: 'preact'
}
```.
所以在 preact 模式下. react 依然可以被找到

官方cli创建的是没有别名的如果这样直接提交可以减少眼瞎的问题

不是的,这个是内置在他们的插件中的,不是显式出现的

https://github.com/NervJS/taro/blob/fa1918dd65530bfa878d7044b634d5b260a56247/packages/taro-plugin-react/src/index.ts#L86

@rhliu1996
Copy link
Contributor Author

可以看下 #47 里的评论.实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {
  react: 'preact'
}
```.
所以在 preact 模式下. react 依然可以被找到

官方cli创建的是没有别名的,如果这样直接提交可以减少眼瞎的问题

不是的,这个是内置在他们的插件中的,不是显式出现的

https://github.com/NervJS/taro/blob/fa1918dd65530bfa878d7044b634d5b260a56247/packages/taro-plugin-react/src/index.ts#L86

找不到我才提交的你可以新建工程测试一下

@innocces
Copy link
Owner

可以看下 #47 里的评论.实际上是不需要主动判断是否安装 preact 的. 因为 taro 内部在 preact 模式下会创建

alias: {

react: 'preact'

}

所以在 preact 模式下. react 依然可以被找到

官方cli创建的是没有别名的,如果这样直接提交可以减少眼瞎的问题

不是的,这个是内置在他们的插件中的,不是显式出现的

https://github.com/NervJS/taro/blob/fa1918dd65530bfa878d7044b634d5b260a56247/packages/taro-plugin-react/src/index.ts#L86

找不到我才提交的你可以新建工程测试一下

好的,晚些我尝试一下

@innocces
Copy link
Owner

确实是的. 项目仓库能找到是因为冒泡到了顶层上.

@innocces innocces merged commit b7c8333 into innocces:next Dec 30, 2023
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep question need code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants