Skip to content

Conversation

tylersayshi
Copy link
Member

prompts => @clack/prompts
kolorist => picocolors

these libraries are copying the pattern set by create-vite

fixes #1289

revival of #1290 since my fork was deleted

Copy link

vercel bot commented Mar 24, 2025

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

Name Status Preview Updated (UTC)
waku ✅ Ready (Inspect) Visit Preview Apr 5, 2025 3:14am
Copy link

codesandbox-ci bot commented Mar 24, 2025

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.

@tylersayshi tylersayshi marked this pull request as draft March 24, 2025 06:25
tylersayshi and others added 2 commits March 29, 2025 21:23
prompts => @clack/prompts
kolorist => picocolors

these libraries are copying the pattern set by create-vite

fixes #1289
@tylersayshi
Copy link
Member Author

this needs to wait for v0.22.0 to be published, but then should be ready. The problem shown here is that waku@latest currently wants react ~19.0 and needs to be ~19.1

image
@tylersayshi
Copy link
Member Author

@dai-shi do you recognize any of the errors in CI? I'm confused where they came from honestly.

@dai-shi
Copy link
Member

dai-shi commented Apr 4, 2025

Some of them are flaky tests: #1249

Others are really failing? Do they work locally?

@tylersayshi
Copy link
Member Author

aha! I am missing the test changes from: https://github.com/wakujs/waku/pull/1290/files

I'll fix that now

@tylersayshi tylersayshi marked this pull request as ready for review April 4, 2025 04:43
tylersayshi and others added 3 commits April 3, 2025 21:56
Co-authored-by: Daishi Kato <dai-shi@users.noreply.github.com>
Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

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

Haven't tested by myself, so 🤞

@dai-shi dai-shi merged commit 944b00e into main Apr 5, 2025
21 checks passed
@dai-shi dai-shi deleted the fix-cli branch April 5, 2025 04:45
@tylersayshi
Copy link
Member Author

@tylersayshi The test seems flaky:

The flakiness is due to installing node modules as part of the test. I'll remove those? They're the only tests that can run long here.

Another option is upping the timeout to 30 or 60 seconds or more per each test to give time for install

@dai-shi
Copy link
Member

dai-shi commented Apr 6, 2025

Are you sure it's due to installing? If it's expected, increasing the timeout to 30 sec sounds good.

@tylersayshi
Copy link
Member Author

I'll check tomorrow. But I do think it's just the install time. npm is slow, maybe I'll have the tests use pnpm somehow too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants