-
Notifications
You must be signed in to change notification settings - Fork 6
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
Re-embed Icon kit @ DevHub #57
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
#echo "Installing root npm" | ||
#pnpm -C $1 i | ||
|
||
#echo "Running root build" | ||
#pnpm -C $1 --filter "*/meteor-icon-kit" build | ||
|
||
#echo "Building figma" | ||
#pnpm -C $1 --filter "*/meteor-icon-kit" start |
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.
Can those be removed?
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'll delete the whole file because the build step is not needed anymore and we can control some stuff from the docs.yml
.
What?
docs:env
,docs:link
anddocs:preview
shortcuts/commands for Icon kitWhy?
So we can switch Icon kit docs to the new monorepo.
How?
Testing?
There's a healthcheck workflow in #51 .
Screenshots (optional)
Anything Else?
We can remove the workflow from
developer-portal
if updatingmeta.json
becomes part of the release process (I see it's part of the repo now).