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

Rebuild fixtures to make build:test-project work properly #5863

Closed
wants to merge 1 commit into from

Conversation

KrisCoulson
Copy link
Contributor

I was receiving multiple errors when trying to run yarn build:test-project after rebuilding the fixtures the errors were no longer happening. @thedavidprice asked me to open a PR with the fixtures updated.

The code below doesn't seem like it updated too much except tailwind to 3.1.4 and reordered a bunch of imports. But it did allow my test project to actually be built correctly.

@netlify
Copy link

netlify bot commented Jul 1, 2022

Deploy Preview for redwoodjs-docs canceled.

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

@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Jul 1, 2022
@jtoar
Copy link
Contributor

jtoar commented Jul 2, 2022

Thanks @KrisCoulson! This looks good, but we should probably merge #5491 first since it also updates the test project. (In this case, you'll probably be the one having to deal with merge conflicts—sorry!)

@jtoar
Copy link
Contributor

jtoar commented Jul 28, 2022

Thanks @KrisCoulson, we rebuilt the fixture for the TS PR (#6033) so I'll close this one out. I'll be better about doing it when it's necessary going forward.

@jtoar jtoar closed this Jul 28, 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: Done
Development

Successfully merging this pull request may close these issues.

2 participants