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

feat: persistent cache expose error to compilation.diagnostic #8765

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

jerrykingxyz
Copy link
Contributor

Summary

Upgrade Cache trait methods allow them to return Result, and compiler will add returned errors to compilation.diagnostic.

Note: SerializeError and DeserializeError are considered unexpected errors and panic directly.

Checklist

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

@github-actions github-actions bot added release: feature release: feature related release(mr only) team The issue/pr is created by the member of Rspack. labels Dec 18, 2024
Copy link

netlify bot commented Dec 18, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit ba3a34e
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/676294b5acf6b20008334c41

@LingyuCoder LingyuCoder enabled auto-merge (squash) December 18, 2024 09:40
@LingyuCoder LingyuCoder merged commit 0e749f0 into main Dec 18, 2024
37 checks passed
@LingyuCoder LingyuCoder deleted the jerry/error branch December 18, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: feature release: feature 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