Skip to content

Update Metro config#6618

Merged
dcalhoun merged 5 commits intoversion-toolkit/gutenberg/build/upgrade-to-react-native-0.73from
build/upgrade-to-react-native-0.73-metro-changes
Feb 8, 2024
Merged

Update Metro config#6618
dcalhoun merged 5 commits intoversion-toolkit/gutenberg/build/upgrade-to-react-native-0.73from
build/upgrade-to-react-native-0.73-metro-changes

Conversation

@geriux
Copy link
Copy Markdown
Contributor

@geriux geriux commented Feb 8, 2024

Related PRs:

To test:

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

Copy link
Copy Markdown
Member

@dcalhoun dcalhoun left a comment

Choose a reason for hiding this comment

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

The Metro server successfully starts with this, so I think we should merge this and iterate on it. Thank you!

projectRoot: path.resolve( __dirname ),
resolver: {
...gutenbergMetroConfig.resolver,
unstable_enableSymlinks: false,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we may be able to keep this enabled. I may follow up with a refactor later.

Comment on lines +33 to +39
// This handles part of the Jetpack Config setup typically handled by Webpack's externals.
if ( moduleName.startsWith( '@automattic/jetpack-config' ) ) {
return {
filePath: path.resolve( __dirname + '/src/jetpack-config.js' ),
type: 'sourceFile',
};
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for addressing this.

@dcalhoun dcalhoun marked this pull request as ready for review February 8, 2024 21:46
@dcalhoun dcalhoun merged commit a73d6ac into version-toolkit/gutenberg/build/upgrade-to-react-native-0.73 Feb 8, 2024
@dcalhoun dcalhoun deleted the build/upgrade-to-react-native-0.73-metro-changes branch February 8, 2024 21:48
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.

2 participants