taro icon indicating copy to clipboard operation
taro copied to clipboard

feat: 优化JSBridge中实例方法过多问题

Open chenai02 opened this issue 1 year ago • 3 comments

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

之前在taro中桥接一个类,需要创建许多类方法进行桥接,本次优化直接统一实例方法,只需要用一个方法即可,不需要过多桥接

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

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

这个 PR 涉及以下平台:

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

chenai02 avatar May 05 '24 11:05 chenai02

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar May 05 '24 11:05 taro-bot2[bot]

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar May 05 '24 11:05 taro-bot2[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.64%. Comparing base (d99ed0c) to head (93b33eb). Report is 6 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #15641   +/-   ##
=======================================
  Coverage   58.64%   58.64%           
=======================================
  Files         500      500           
  Lines       21936    21936           
  Branches     5621     5615    -6     
=======================================
  Hits        12865    12865           
- Misses       7906     7933   +27     
+ Partials     1165     1138   -27     
Flag Coverage Δ
taro-cli 75.00% <ø> (+17.59%) :arrow_up:
taro-runner 54.00% <ø> (+7.68%) :arrow_up:
taro-runtime 79.39% <ø> (+14.04%) :arrow_up:
taro-web 44.71% <ø> (+4.08%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

codecov[bot] avatar May 05 '24 11:05 codecov[bot]