Skip to content

[chore] Fix release script copies invalid node_modules subfolder#11038

Merged
djhi merged 1 commit intomasterfrom
fix-release-script
Nov 13, 2025
Merged

[chore] Fix release script copies invalid node_modules subfolder#11038
djhi merged 1 commit intomasterfrom
fix-release-script

Conversation

@slax57
Copy link
Contributor

@slax57 slax57 commented Nov 13, 2025

Problem

Release script copies everything inside the package folders into EE node_modules. This causes an issue because it also copies a folder under node_modules/react-admin/node_modules/ra-ui-materialui, which causes a weird TS error when building ra-rbac about components not being portable.

Solution

Also exclude this folder from the copy (it should not be there when installing the package from npm anyway)

How To Test

Run step 3 of the release script, then attempt to build ra-rbac make build-ra-rbac.

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature

Also, please make sure to read the contributing guidelines.

@djhi djhi added this to the 5.13.1 milestone Nov 13, 2025
@djhi djhi merged commit 63b6405 into master Nov 13, 2025
2 checks passed
@djhi djhi deleted the fix-release-script branch November 13, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RFR Ready For Review

Development

Successfully merging this pull request may close these issues.

2 participants