-
Notifications
You must be signed in to change notification settings - Fork 65
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
chore(examples): Does some cleanup and some dep updates #343
Conversation
3443cb4
to
768753b
Compare
🚀 PR was released in |
@@ -19,6 +19,7 @@ | |||
"compile": "babel src --out-dir dist -s --source-maps --extensions '.ts,.tsx' --ignore src/**/__tests__", | |||
"kitchen-sink": "cd examples/kitchen-sink && yarn start", | |||
"lint": "tslint -c tslint.json --project tsconfig.json", | |||
"postinstall": "yarn compile", |
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.
not sure if it is just me but got some error on installing the latest package
[8/14] ⠐ @apollo/rover
warning Error running install script for optional dependency: "/Users/roderickhsiao/git/kindred-landing/node_modules/cpu-features: Command failed.
Exit code: 127
Command: node buildcheck.js > buildcheck.gypi && node-gyp rebuild
Arguments:
[7/14] ⠄ sharp
[11/14] ⠄ cypress
[13/14] ⠄ ssh2
[8/14] ⠄ @apollo/rover
error /Users/roderickhsiao/git/kindred-landing/node_modules/@artsy/fresnel: Command failed.
Exit code: 127
Command: yarn compile
Arguments:
Directory: /Users/roderickhsiao/git/kindred-landing/node_modules/@artsy/fresnel
Output:
yarn run v1.16.0
warning ../../../../package.json: No license field
$ babel src --out-dir dist -s --source-maps --extensions '.ts,.tsx' --ignore src/**/__tests__
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.
Ah, lemme fix that
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.
Fixed in #345
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.
Lightening speed thank you!
Noticed we were getting errors outside of Node 16, so upgraded webpack all around. Also pointed example fresnel package ref to internal code.