Skip to content

fix(core): modifying the dev.watchFiles config in plugins is not effective#4504

Merged
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
xbzhang2020:fix-dev-watchfiles
Feb 8, 2025
Merged

fix(core): modifying the dev.watchFiles config in plugins is not effective#4504
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
xbzhang2020:fix-dev-watchfiles

Conversation

@xbzhang2020
Copy link
Copy Markdown
Contributor

@xbzhang2020 xbzhang2020 commented Feb 8, 2025

Summary

Fix the issue that modifying the dev.watchFiles config in plugins is not effective.

In source code, the loadConfig method can't resolve the config in plugins, which can be resolved by rsbuild.getNormalizedConfig(). So changing the occasion for loading dev.watchFiles will fix the problem.

Related Links

related issue #4503

Checklist

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

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 8, 2025

Deploy Preview for rsbuild ready!

Name Link
🔨 Latest commit f3e79ca
🔍 Latest deploy log https://app.netlify.com/sites/rsbuild/deploys/67a6b946e851290008746537
😎 Deploy Preview https://deploy-preview-4504--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: 67 (🔴 down 5 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 site configuration.

@xbzhang2020 xbzhang2020 changed the title fix(core): Modify the dev.watchFiles in plugin is not effective fix(core): modifying the dev.watchFiles in plugin is not effective Feb 8, 2025
@xbzhang2020 xbzhang2020 changed the title fix(core): modifying the dev.watchFiles in plugin is not effective fix(core): modifying the dev.watchFiles config in plugin is not effective Feb 8, 2025
@xbzhang2020 xbzhang2020 changed the title fix(core): modifying the dev.watchFiles config in plugin is not effective fix(core): modifying the dev.watchFiles config in plugins is not effective Feb 8, 2025
Copy link
Copy Markdown
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.

LGTM, thank you!

@chenjiahan chenjiahan enabled auto-merge (squash) February 8, 2025 02:13
@chenjiahan chenjiahan merged commit 55fda73 into web-infra-dev:main Feb 8, 2025
14 checks passed
@xbzhang2020 xbzhang2020 deleted the fix-dev-watchfiles branch February 8, 2025 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants