Skip to content

fix: unify js assets info type#9464

Merged
SyMind merged 8 commits intomainfrom
fix-js-asset-info
Feb 26, 2025
Merged

fix: unify js assets info type#9464
SyMind merged 8 commits intomainfrom
fix-js-asset-info

Conversation

@SyMind
Copy link
Member

@SyMind SyMind commented Feb 26, 2025

Summary

fix: unit js assets info type

KnownAssetInfo and AssetInfo currently defined by napi.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack. labels Feb 26, 2025
@netlify
Copy link

netlify bot commented Feb 26, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit 7ff002e
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67be8f519c68a000095e499f
😎 Deploy Preview https://deploy-preview-9464--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 26, 2025

CodSpeed Performance Report

Merging #9464 will not alter performance

Comparing fix-js-asset-info (7ff002e) with main (9a6473c)

Summary

✅ 7 untouched benchmarks

Copy link
Member

@chenjiahan chenjiahan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related documentation should be updated too:

image

@chenjiahan chenjiahan changed the title fix: unit js assets info type fix: unify js assets info type Feb 26, 2025
@SyMind SyMind merged commit 1f85fa0 into main Feb 26, 2025
29 checks passed
@SyMind SyMind deleted the fix-js-asset-info branch February 26, 2025 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants