-
Notifications
You must be signed in to change notification settings - Fork 25
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: upgrade all the deps #319
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 1234d9e:
|
@@ -32,10 +32,10 @@ | |||
"@storybook/testing-library": "^0.0.13", | |||
"@types/react": "^18.0.26", | |||
"@types/react-dom": "^18.0.9", | |||
"@ucanto/interface": "^4.0.3", | |||
"@ucanto/interface": "^4.2.3", |
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.
I think it would be a good idea for @w3ui/react
and perhaps others from the family to re-export same basics like @ucanto/interaface
and perhaps some things from multiformats
so that they don't need to be imported by the users and deal with dependency problems.
@@ -1,5 +1,5 @@ | |||
import React from 'react' | |||
import { CID } from 'multiformats/cid' | |||
import * as Link from 'multiformats/link' |
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.
Note: CID still exists but Link
is designed to be more abstract and less prone to becoming out of date.
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.
looks good to me!
Updates all the deps and aligns versions so we don't have various version of the multiformat and @ipld/car
Most updates have been generate with a help of
pnpm upgrade --interactive -r
command, however for some reason it was failing to update deps inexamples
directory resulting in multiple versions ofmultiformats
and@ipld/car
. I end up working around this by manually changingpackage.json
-s of packages that were referencing old versions.