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

feat(crwa): Always install using yarn v4 #9861

Merged
merged 2 commits into from
Jan 21, 2024

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented Jan 21, 2024

By cd-ing into the new rw app directory we make sure yarn install is executed by yarn v4 (or whatever version is set in the root package.json)

This has a somewhat of a troubled past.
#9859
#9786 (comment)
#8164

@Tobbe Tobbe changed the title feat(crwa): Offer to install with all yarn versions feat(crwa): Offer to install no matter what yarn version the user has installed Jan 21, 2024
@Tobbe Tobbe changed the title feat(crwa): Offer to install no matter what yarn version the user has installed feat(crwa): Always install using yarn v4 Jan 21, 2024
@Tobbe Tobbe added the release:feature This PR introduces a new feature label Jan 21, 2024
@Tobbe Tobbe added this to the next-release milestone Jan 21, 2024
@Tobbe Tobbe merged commit 6f38084 into redwoodjs:main Jan 21, 2024
37 of 42 checks passed
@Tobbe Tobbe deleted the tobbe-crwa-install branch January 21, 2024 21:30
jtoar added a commit that referenced this pull request Jan 21, 2024
This PR fixes the CRWA e2e tests after the functionality introduced in
#9861.

For now I'm skipping `yarn install` since it's prone to timeouts in CI,
and the subsequent step, generating types, is also flakey since `yarn
pack` seems to skip the `.yarnrc.yml` which is necessary for configuring
a proper install.
jtoar pushed a commit that referenced this pull request Jan 22, 2024
The check is currently running indefinitely as it is waiting for the
result of a prompt. This PR passes the `--no-yarn-install` flag to
prevent this. That flag was introduced in #9861
dac09 added a commit that referenced this pull request Jan 22, 2024
…edwood into try/setupScenario-multiple-tests

* 'try/setupScenario-multiple-tests' of github.com:dac09/redwood: (235 commits)
  chore(project-config): switch to vitest (#9864)
  chore(ci): Fix CRWA telemetry check (#9865)
  chore(crwa): update e2e test to handle yarn install patch (#9862)
  feat(crwa): Always install using yarn v4 (#9861)
  fix(cli): Skip Studio when running `rw upgrade` (#9860)
  fix(studio): Upgrade to 11.0.1 and enable web-side (#9858)
  feat(server file): add `createServer` (#9845)
  chore(crwa): set `REDWOOD_CI` and `REDWOOD_DISABLE_TELEMETRY` (#9857)
  Fix(crwa): Exit 0 after Quit install (#9856)
  chore(crwa): switch to vitest (#9855)
  chore(api): Switch to use vitest over jest (#9853)
  fix(server): ensure consistency between CLI serve entrypoints regarding help and strict (#9809)
  Improve how the api-server watch command works (#9841)
  docs(typo): correct grammar in realtime docs (#9850)
  Add support for Prisma Bytes and GraphQL scalar Byte (#9847)
  packages/cli: Switch from substr (deprecated) to slice (#9849)
  mapPrismaScalarToPagePropTsType: Fix Decimal test case (#9846)
  chore(build): Avoid prebuilding api side, instead use an esbuild plugin (#9767)
  Fix url query param encoding (#9844)
  chore(fwtools): handle existing resolutions in tarsync (#9840)
  ...
jtoar pushed a commit that referenced this pull request Jan 24, 2024
jtoar pushed a commit that referenced this pull request Jan 24, 2024
jtoar added a commit that referenced this pull request Jan 24, 2024
This PR fixes the CRWA e2e tests after the functionality introduced in
#9861.

For now I'm skipping `yarn install` since it's prone to timeouts in CI,
and the subsequent step, generating types, is also flakey since `yarn
pack` seems to skip the `.yarnrc.yml` which is necessary for configuring
a proper install.
jtoar pushed a commit that referenced this pull request Jan 24, 2024
The check is currently running indefinitely as it is waiting for the
result of a prompt. This PR passes the `--no-yarn-install` flag to
prevent this. That flag was introduced in #9861
jtoar added a commit that referenced this pull request Jan 25, 2024
@jtoar jtoar modified the milestones: next-release-patch, v6.6.3 Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:feature This PR introduces a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants