Skip to content

chore: get clayui.com site working on vercel #6101

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bryceosterhaus
Copy link
Member

@bryceosterhaus bryceosterhaus commented Jul 10, 2025

These commits get the clay site working with auto-builds on vercel(tested on my own account).

I am just waiting on legal and IT to get the proper vercel account so that we can link the domain.

Note: we may need to tweak lerna when we release next after merging this. I tested it a bit, but not a full publish.

@ethib137
Copy link
Member

If we are upgrading node in this PR can we close: #6099 ?

@pat270
Copy link
Member

pat270 commented Jul 11, 2025

I'm getting an error when running yarn lerna.

$ lerna bootstrap -- --no-optional --no-package-lock
ERR! bootstrap The "bootstrap" command was removed by default in v7, and is no longer maintained.
ERR! bootstrap Learn more about this change

@bryceosterhaus
Copy link
Member Author

@pat270 do you use that command specifically for anything? I noticed we don't use yarn lerna in our release process, so I wonder if we can just remove that script. I believe yarn with workspaces supersedes lerna bootstrap

@ethib137
Copy link
Member

If we don't need the lerna bootstrap command anymore we should make sure to remove the script from our package.json. We would also need to update our contributing docs which recommend the use of that command.

@pat270
Copy link
Member

pat270 commented Jul 14, 2025

@bryceosterhaus I was following the instructions on the docs. I have no idea what we use lerna for. I assumed it had to do with running Clay in a dev environment.

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

Successfully merging this pull request may close these issues.

3 participants