Skip to content

fix: use readFileSync instead of require in validatePlugins#1143

Merged
jbroma merged 2 commits into
mainfrom
fix/validate-plugins-read
May 4, 2025
Merged

fix: use readFileSync instead of require in validatePlugins#1143
jbroma merged 2 commits into
mainfrom
fix/validate-plugins-read

Conversation

@jbroma
Copy link
Copy Markdown
Contributor

@jbroma jbroma commented May 4, 2025

Summary

  • - fixed behaviour of validatePlugins where it fails to read package.json when node version has support for require(esm)

Test plan

  • - testers work

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 4, 2025

🦋 Changeset detected

Latest commit: 6c30574

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@callstack/repack Patch
@callstack/repack-plugin-expo-modules Patch
@callstack/repack-plugin-nativewind Patch
@callstack/repack-plugin-reanimated Patch
@callstack/repack-dev-server Patch
@callstack/repack-init Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented May 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
repack-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 4, 2025 8:11pm

@jbroma jbroma changed the title fix: use readFileSync instead of require in validatePlugins fix: use readFileSync instead of require in validatePlugins May 4, 2025
@jbroma jbroma merged commit 81ee94c into main May 4, 2025
5 checks passed
@jbroma jbroma deleted the fix/validate-plugins-read branch May 4, 2025 20:15
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