Skip to content

Conversation

@AJLemos
Copy link
Contributor

@AJLemos AJLemos commented Nov 6, 2025

While following the CONTRIBUTING.md guide I ran into a few issues running yarn commands until I took a look at package.json and saw the repo now uses pnpm (great choice btw).

The E2E command still needs to be updated but I haven't worked through installing and running cypress yet so I wasn't sure what the correct documentation there should be yet.

The repo was updated to use pnpm here:
rpldy#611

The repo was updated to use vitest here:
rpldy#604
@yoavniran
Copy link
Collaborator

Thanks @AJLemos !
I totally forgot about updating it so thank you for the PR 😆

About E2E, its a good point, I should add that too. It's a little involved and I think there are a couple of specs that dont pass without special flags. I need to sit myself down and document it thoroughly.

Co-authored-by: Yoav Niran <yoavniran@users.noreply.github.com>
@yoavniran yoavniran merged commit 941df82 into rpldy:master Nov 11, 2025
7 of 8 checks passed
@github-actions
Copy link
Contributor

📦 Bundle Size Report

namesizemaxsuccess
ui-core-chunked15.96KB16.6KB🟢
core10.59KB10.74KB🟢
ui-core13.54KB14.65KB🟢
all23.76KB24.41KB🟢

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.

2 participants