Skip to content

Conversation

dai-shi
Copy link
Member

@dai-shi dai-shi commented Dec 8, 2024

Finally!

It's unfortunate that Waku isn't v1 yet.

Copy link

vercel bot commented Dec 8, 2024

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
waku ⬜️ Ignored (Inspect) Visit Preview Dec 8, 2024 11:08pm

Copy link

codesandbox-ci bot commented Dec 8, 2024

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.

@dai-shi dai-shi mentioned this pull request Dec 8, 2024
@dai-shi dai-shi marked this pull request as ready for review December 8, 2024 12:00
@tylersayshi
Copy link
Member

Should we keep any of our tests on canary?

@dai-shi
Copy link
Member Author

dai-shi commented Dec 8, 2024

Should we keep any of our tests on canary?

I thought about it too, but we don't want to increase CI time for each PR.
I will probably create a separate PR so that we can run tests on canary manually, on demand.

@dai-shi dai-shi merged commit 40a9542 into main Dec 9, 2024
26 checks passed
@dai-shi dai-shi deleted the react-19-stable branch December 9, 2024 00:06
@tylersayshi
Copy link
Member

Should we keep any of our tests on canary?

I thought about it too, but we don't want to increase CI time for each PR. I will probably create a separate PR so that we can run tests on canary manually, on demand.

we could run a github action exclusively on main to test canary versions.

This would mean that PRs would have the same pipeline time, but we run an additional check after it is merged to main, so we should know of issues on canary before we go go do a release.

If it is its own CI process we can re-use the same tests too and just run the pnpm update to canary on the CI job.

I'm happy to work on a PR for this if it sounds good to you

@dai-shi
Copy link
Member Author

dai-shi commented Dec 9, 2024

we could run a github action exclusively on main to test canary versions.

good idea.

I'm happy to work on a PR for this if it sounds good to you

Yeah, please go ahead!

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