Skip to content
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

feat: add syntax highlighting to homepage #140

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nandorojo
Copy link

I couldn't preview it locally without adding a bunch of env variables, but assuming Vercel generates a preview link of this PR, we can preview it there.

Uses bright for RSC-compatible syntax-highlighting.

Copy link

vercel bot commented Dec 24, 2024

@nandorojo is attempting to deploy a commit to the Antiwork Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Dec 24, 2024

CLA assistant check
All committers have signed the CLA.

@m2rads
Copy link
Contributor

m2rads commented Dec 24, 2024

@nandorojo Could you please provide a screenshot of how it looks now?

@nandorojo
Copy link
Author

Yeah I want to except the app crashes when I try to run it since I don't have env variables set. Specifically clerk throws

@m2rads
Copy link
Contributor

m2rads commented Dec 24, 2024

Can you try running the setup script in package.json? That should help you to add the required env vars. For clerk you would need these two:

NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=
CLERK_SECRET_KEY=

All the others are explained in .env.example. Let me know if you have any questions.

Copy link

vercel bot commented Dec 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
shortest ❌ Failed (Inspect) Dec 24, 2024 9:58pm

@m2rads
Copy link
Contributor

m2rads commented Dec 24, 2024

There is a build issue in preview env:

image

@nandorojo
Copy link
Author

Hm yeah I wonder if I can just use fake clerk creds let me try this

@slavingia
Copy link
Contributor

^ if that works maybe we provide them by default so people can easily run the website without needing to setup Clerk.

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.

4 participants