Skip to content
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

Merged
merged 3 commits into from
Nov 2, 2023

Conversation

damassi
Copy link
Member

@damassi damassi commented Nov 2, 2023

Noticed we were getting errors outside of Node 16, so upgraded webpack all around. Also pointed example fresnel package ref to internal code.

@damassi damassi self-assigned this Nov 2, 2023
@damassi damassi merged commit f634d9a into main Nov 2, 2023
7 checks passed
@damassi damassi deleted the damassi/chore/example-deps branch November 2, 2023 17:40
@artsyit
Copy link
Contributor

artsyit commented Nov 2, 2023

🚀 PR was released in v7.1.0 🚀

@@ -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",
Copy link
Contributor

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__

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, lemme fix that

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed in #345

Copy link
Contributor

Choose a reason for hiding this comment

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

Lightening speed thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants