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

Migrate codebase to pnpm + integrate prism react renderer 2 #354

Merged
merged 10 commits into from
May 10, 2023

Conversation

carloskelly13
Copy link
Contributor

@carloskelly13 carloskelly13 commented May 5, 2023

  • Migrates to pnpm workspaces
  • Moves to tsup for building the library
  • Migrates to Prism React Renderer 2
  • Fixes various typing issues

Also gets doc site previews → https://react-live-docs-website-git-task-pnpm-formidable-labs.vercel.app/

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@carloskelly13 carloskelly13 requested a review from gksander May 5, 2023 14:30
@FormidableLabs FormidableLabs deleted a comment from vercel bot May 5, 2023
@vercel
Copy link

vercel bot commented May 5, 2023

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

Name Status Preview Comments Updated (UTC)
react-live-docs-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2023 0:10am

Copy link
Contributor

@gksander gksander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, a few comments overall.

"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-live": "^4.0.0",
"prism-react-renderer": "*",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be an explicit version, since PRR isn't part of this workspace?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It isn't, but the * also means, I believe, use the version specified in the root package.json too.

Co-authored-by: Grant Sander <gksander93@gmail.com>
Copy link
Contributor

@gksander gksander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, this fell off my radar.

@carloskelly13 carloskelly13 merged commit eea1186 into master May 10, 2023
@carloskelly13 carloskelly13 deleted the task/pnpm branch May 10, 2023 16:00
@github-actions github-actions bot mentioned this pull request May 10, 2023
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.

None yet

2 participants