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

chore(ci): use canary version of rwjs vite #8526

Merged
merged 5 commits into from
Jun 6, 2023
Merged

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Jun 6, 2023

#8471 added a new binary to a redwoodjs package. Our current tooling (yarn rwfw project:deps, copy sync) alone doesn't handle this use case. Instead we need the package with the binary to be installed, so I'm bumping @redwoodjs/vite to canary to see if it fixes it. If it does, I'll have to add a step to rebuilding the test project to persist this fix till we release the next major.

@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Jun 6, 2023
@dac09
Copy link
Collaborator

dac09 commented Jun 6, 2023

Looks like multiple reasons why CI was failures, and you got them all here @jtoar!

✅ Cypress wasn't being installed 🤷
✅ Disable replay because it seems to be broken
✅ Bump vite to canary manaully

@dac09 dac09 merged commit 8a0adbc into main Jun 6, 2023
25 checks passed
@dac09 dac09 deleted the ds-ci/try-rwjs-vite-canary branch June 6, 2023 05:22
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 6, 2023
@jtoar jtoar modified the milestones: next-release, v6.0.0 Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants