-
Notifications
You must be signed in to change notification settings - Fork 31
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
Cleanup dependencies #1430
base: main
Are you sure you want to change the base?
Cleanup dependencies #1430
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
9cb20cb
to
1c8778e
Compare
1c8778e
to
a6fe94e
Compare
a6fe94e
to
987862c
Compare
@@ -71,6 +71,7 @@ | |||
}, | |||
"devDependencies": { | |||
"@cartridge/eslint": "workspace:^", | |||
"@cartridge/presets": "catalog:", |
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.
is this needed?
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 is used in storybook for mainly layout thus devDependencies.
import { defaultTheme } from "@cartridge/presets"; |
@@ -28,6 +28,7 @@ | |||
"viem": "catalog:" | |||
}, | |||
"devDependencies": { | |||
"@cartridge/presets": "catalog:", |
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.
is this needed?
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 is used to pass ERC20 icons to ERC20 class. In the future, it should be provided via gateway API.
@@ -30,9 +30,6 @@ | |||
"test:storybook": "pnpm turbo build:deps test:storybook", | |||
"test:storybook:update": "pnpm turbo build:deps test:storybook:update" | |||
}, | |||
"dependencies": { | |||
"@cartridge/presets": "github:cartridge-gg/presets#c7e6fb7" |
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.
we need to make sure this change works with the presets update ci
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 point !
987862c
to
e6649e1
Compare
e6649e1
to
4b276bc
Compare
Related