Skip to content

chore(cache): change cache enabled message to debug log#5580

Merged
chenjiahan merged 1 commit intomainfrom
cache_debug_0711
Jul 11, 2025
Merged

chore(cache): change cache enabled message to debug log#5580
chenjiahan merged 1 commit intomainfrom
cache_debug_0711

Conversation

@chenjiahan
Copy link
Member

Summary

Updated the log level to debug for the message indicating that Rspack persistent cache is enabled, since the Rspack persistent cache is now more stable.

Checklist

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

Copilot AI review requested due to automatic review settings July 11, 2025 04:52
@netlify
Copy link

netlify bot commented Jul 11, 2025

Deploy Preview for rsbuild ready!

Name Link
🔨 Latest commit 66a5e42
🔍 Latest deploy log https://app.netlify.com/projects/rsbuild/deploys/68709878bc333c00084862f1
😎 Deploy Preview https://deploy-preview-5580--rsbuild.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 74 (no change from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adjusts logging for the Rspack persistent cache message by lowering its level to debug and removing the “experimental” label, and cleans up related imports.

  • Removed color import since it’s no longer used.
  • Changed the cache-enabled message from logger.info to logger.debug and dropped the experimental suffix.
Comments suppressed due to low confidence (2)

packages/core/src/plugins/cache.ts:181

  • Add or update tests to ensure the log level change from info to debug and removal of the experimental label for the Rspack persistent cache.
        logger.debug('Rspack persistent cache enabled');

packages/core/src/plugins/cache.ts:180

  • Consider updating documentation comments or external docs to reflect that Rspack persistent cache is now considered stable and no longer experimental.
      if (cacheEnabled && api.context.bundlerType === 'rspack') {

@chenjiahan chenjiahan merged commit 9458b51 into main Jul 11, 2025
13 checks passed
@chenjiahan chenjiahan deleted the cache_debug_0711 branch July 11, 2025 04:59
@chenjiahan chenjiahan mentioned this pull request Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant