-
-
Notifications
You must be signed in to change notification settings - Fork 496
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
fix: should not provide for defined variable #5489
Conversation
Test Compatibility Diff
Unpassed tests1 ⚪️ Examples aggressive-merging: TODO |
Summary
before:
This PR also align the behavior of
member()
/get_member_expression_info()
The only missing test case of ProvidePlugin is
function() { this.env.NODE_ENV }.call(process)
case, this need supportwalkIIFE
Test Plan
update
packages/rspack/tests/configCases/builtins/provide
Require Documentation?