Skip to content

Conversation

@ZakaryCode
Copy link
Contributor

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

  • 修复 Harmony 端 asset 导出地址错误
  • 优化 vite resolve multi 文件逻辑

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

  • 错误修复(Bugfix) issue: fix #

这个 PR 涉及以下平台:

  • 鸿蒙(harmony)
  • Vite runner

@ZakaryCode ZakaryCode requested a review from ZEJIA-LIU November 6, 2024 07:12
@ZakaryCode ZakaryCode added A-runner Area - Webpack runner 相关 T-harmony Target - 编译到 Harmony labels Nov 6, 2024
@codecov
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.50%. Comparing base (6ed8d05) to head (b586f9e).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             main   #16815    +/-   ##
========================================
  Coverage   58.50%   58.50%            
========================================
  Files         466      466            
  Lines       27107    27107            
  Branches     5946     5737   -209     
========================================
  Hits        15860    15860            
- Misses       9769     9887   +118     
+ Partials     1478     1360   -118     
Flag Coverage Δ
taro-cli 73.67% <ø> (ø)
taro-runtime 59.87% <ø> (ø)
taro-web 52.68% <ø> (ø)

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

see 39 files with indirect coverage changes

@ZakaryCode ZakaryCode merged commit 9a6bf39 into main Nov 6, 2024
42 checks passed
@ZakaryCode ZakaryCode deleted the feat/harmony branch November 6, 2024 08:14
This was referenced Nov 9, 2024
tutuxxx pushed a commit that referenced this pull request Nov 29, 2024
* fix(vite): 修复 asset 路径错误

* fix(vite): 优化 multi 查询逻辑

* chore(release): publish 4.0.8-beta.0 --tag=beta
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-runner Area - Webpack runner 相关 T-harmony Target - 编译到 Harmony

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants