-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Update package.json to make the dist avaliable again #23991
base: main
Are you sure you want to change the base?
Conversation
js/react_native/package.json
Outdated
"bootstrap": "npm run pack-common && npm run unpack-common && npm run pack-libs && npm run unpack-libs && npm run e2e && npm run pods", | ||
"test": "jest", | ||
"pack-common": "cd ../common && npm pack && mv -f onnxruntime-common-*.tgz ../react_native/e2e/onnxruntime-common.tgz", | ||
"unpack-common": "cd e2e && npm install --no-save ./onnxruntime-common.tgz", | ||
"unpack-common": "cd e2e && npm install ./onnxruntime-common.tgz", |
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.
npm install --no-save doesn't update package-lock.json nor package.json
"dependencies": { | ||
"buffer": "^6.0.3" | ||
}, | ||
"scripts": { | ||
"typescript": "tsc --noEmit", | ||
"bootstrap": "npm run pack-common && npm run unpack-common && npm run pack-libs && npm run unpack-libs && npm run e2e && npm run pods", | ||
"prepare": "npx bob build", | ||
"bootstrap-no-pods": "npm run pack-common && npm run unpack-common && npm run pack-libs && npm run unpack-libs && npm run e2e", |
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.
Create this script so both Android and iOS can use it.
@@ -62,44 +61,7 @@ jobs: | |||
targetFolder: $(Build.SourcesDirectory)/js/react_native/android/libs | |||
displayName: Copy Android package to React Native directory | |||
|
|||
- script: | |
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.
This has been moved to react-native-bootstrap-steps.yml
Description
Adding back missing dist folder
Motivation and Context