Conversation
|
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/chakra-ui/chakra-ui/57z5pwhyY39qiPyTSSyKtQV6G9EX [Deployment for f9a3771 failed] |
with-heart
left a comment
There was a problem hiding this comment.
This is a nice little improvement to our workflows. I like the idea of removing the gen commands from the docs build itself as it's not really a direct concern.
| "docs:deploy": "yarn build && yarn docs:build-ci && cp -r website/public .", | ||
| "docs:build-ci": "yarn members:gen && yarn docs:build", | ||
| "docs:build": "yarn avatars:gen && yarn search-meta:gen && yarn docs build", | ||
| "docs:dev": "yarn avatars:gen && yarn docs dev", | ||
| "docs:build-ci": "yarn docs:build", |
There was a problem hiding this comment.
Could you remove the docs:build-ci script? We can just use yarn docs:build in docs:deploy now.
|
@allcontributors please add @tomchentw for infra |
|
I've put up a pull request to add @tomchentw! 🎉 |
599fd98 to
f9a3771
Compare
|
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 f9a3771:
|
|
Thank you! |
📝 Description
Runs these two scripts daily on GitHub Actions and commit the results to the repository:
yarn avatars:genyarn members:gen⛳️ Current behavior (updates)
These scripts needs to be run by parts of the build process.
🚀 New behavior
Runs these two scripts daily on GitHub Actions. This saves time for local development as well as reducing the API usage.