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): Fix rebuild fixture #6033

Merged
merged 6 commits into from
Jul 25, 2022
Merged

Conversation

dac09
Copy link
Contributor

@dac09 dac09 commented Jul 25, 2022

Rebuild fixture will now copy over framework dependencies (just like --link) when generating a test project.

At the end just before replacing the test-project in __fixtures/test-project it'll copy over the package.json from the crwa template, to make sure we don't have the linked devDeps in the fixture.

@nx-cloud
Copy link

nx-cloud bot commented Jul 25, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 62c56bc. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 14 targets

Sent with 💌 from NxCloud.

@netlify
Copy link

netlify bot commented Jul 25, 2022

Deploy Preview for redwoodjs-docs canceled.

Name Link
🔨 Latest commit 62c56bc
🔍 Latest deploy log https://app.netlify.com/sites/redwoodjs-docs/deploys/62deab52c54db80009b18ffa

@dac09 dac09 added the release:chore This PR is a chore (means nothing for users) label Jul 25, 2022
dac09 and others added 3 commits July 25, 2022 13:50
playwright has many kinds of timeouts. I've seen this test fail on both "timeouts" and "expect timeouts". we've increased timeouts in the past, but never expect timeouts, so let's try increasing those (from 5 seconds to 10 seconds). I also increased the regular timeouts for linux + macOs cause I've seen it fail at 60s too
@dac09 dac09 merged commit 1125d9f into redwoodjs:main Jul 25, 2022
@dac09 dac09 deleted the fix/rebuild-fixture-with-deps branch July 25, 2022 15:05
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jul 25, 2022
dac09 added a commit to dac09/redwood that referenced this pull request Jul 25, 2022
…ctmode-gen

* 'main' of github.com:redwoodjs/redwood:
  (chore): Fix rebuild fixture (redwoodjs#6033)
@jtoar jtoar modified the milestones: next-release, v3.0.0 Sep 2, 2022
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
Status: Archived
Development

Successfully merging this pull request may close these issues.

2 participants