Conversation
|
| @@ -133,7 +133,7 @@ | |||
| "scripts/xcode/ccache.conf", | |||
| "scripts/xcode/with-environment.sh", | |||
| "sdks/.hermesversion", | |||
| "sdks/hermes-engine", | |||
| "sdks/hermes-engine/**", | |||
There was a problem hiding this comment.
Turns out this is why yarn npm publish skipped a bunch of files.. I would like to upstream this to React Native since it's compatible with npm
| return true; | ||
| } | ||
|
|
||
| async function checkForMajorBumps() { |
There was a problem hiding this comment.
This file is mostly copied from FURN, we probably should adapt the checks to RNMs requirements
Summary:
nx releasehas proven to be a bit more cumbersome than we like. I just migrated FluentUI React Native to changesets (microsoft/fluentui-react-native#4003), let's do the same here. We can adopt some of the changes from #2807 , specifically around updatingprepublish-check.mjsinto an all-in-one script, now named,configure-publish.mts.Test Plan:
There should be an updated