-
Notifications
You must be signed in to change notification settings - Fork 63
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
Migrate to npm #539
Migrate to npm #539
Conversation
b35c6bd
to
7e57c4f
Compare
@@ -14,18 +14,7 @@ const modules = Object.keys({...pak.peerDependencies}); | |||
* @type {import('metro-config').MetroConfig} | |||
*/ | |||
const config = { | |||
watchFolders: [root], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -11,9 +11,9 @@ react { | |||
// The root of your project, i.e. where "package.json" lives. Default is '../..' | |||
// root = file("../../") | |||
// The folder where the react-native NPM package is. Default is ../../node_modules/react-native | |||
// reactNativeDir = file("../../node_modules/react-native") | |||
reactNativeDir = file("../../../node_modules/react-native") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -56,7 +56,7 @@ jobs: | |||
example/ios/Pods | |||
~/Library/Caches/CocoaPods | |||
~/.cocoapods | |||
key: build-ios-pods-${{ matrix.react-native-architecture }}-${{ hashFiles('example/node_modules/react-native/package.json') }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WebExample/App.tsx
Outdated
@@ -1,3 +1,4 @@ | |||
import App from '../example/src/App'; | |||
import App from '@expensify/react-native-live-markdown-example/src/App'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fast refresh is broken because of this change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job on this PR! Thanks! 👏
Details
Related Issues
GH_LINK
Manual Tests
Linked PRs