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

taro rn 支持 config/index.ts 文件 #14201

Merged
merged 10 commits into from
Aug 6, 2023
Merged

taro rn 支持 config/index.ts 文件 #14201

merged 10 commits into from
Aug 6, 2023

Conversation

zhiqingchen
Copy link
Member

@zhiqingchen zhiqingchen commented Jul 18, 2023 •
edited
Loading

这个 PR 做了什么? (简要描述所做更改)

  1. 支持 config/index.ts 文件
  2. 支持新版的异步配置
  3. 支持 .env 配置环境变量

注意事项

  1. 新版本的 metro.config.js 模板有变更。

这个 PR 是什么类型? (至少选择一个)

  • 错误修复(Bugfix) issue: fix #
  • 新功能(Feature)
  • 代码重构(Refactor)
  • TypeScript 类型定义修改(Typings)
  • 文档修改(Docs)
  • 代码风格更新(Code style update)
  • 其他，请描述(Other, please describe):

这个 PR 涉及以下平台:

  • 所有小程序
  • 微信小程序
  • 支付宝小程序
  • 百度小程序
  • 字节跳动小程序
  • QQ 轻应用
  • 京东小程序
  • 快应用平台（QuickApp）
  • Web 平台（H5）
  • 移动端（React-Native）
  • 鸿蒙（harmony）

@zhiqingchen
Copy link
Member Author

zhiqingchen commented Jul 18, 2023 •
edited
Loading

@iambool 3.6.11-alpha.0

@zhiqingchen zhiqingchen marked this pull request as draft July 21, 2023 07:01
@taro-bot2 taro-bot2 bot mentioned this pull request Jul 22, 2023
Closed
@zhiqingchen zhiqingchen marked this pull request as ready for review July 28, 2023 09:48
@zhiqingchen
Copy link
Member Author

@bigmeow @Chen-jj

dotenv 相关代码挪到了 helper 中，方便 RN 调用

@zhiqingchen zhiqingchen requested a review from biorz August 3, 2023 09:04
@zhiqingchen
Copy link
Member Author

@bigmeow 可以继续review了

@taro-bot2 taro-bot2 bot mentioned this pull request Aug 5, 2023
Closed
@zhiqingchen zhiqingchen merged commit 14bf92c into NervJS:next Aug 6, 2023
4 checks passed
@zhiqingchen zhiqingchen mentioned this pull request Aug 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bigmeow bigmeow bigmeow left review comments

@biorz biorz biorz approved these changes

@yechunxi yechunxi yechunxi approved these changes

@Chen-jj Chen-jj Awaiting requested review from Chen-jj

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zhiqingchen @yechunxi @bigmeow @biorz