-
Notifications
You must be signed in to change notification settings - Fork 8
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
Swap DrupalVM for ddev. #4458
Swap DrupalVM for ddev. #4458
Conversation
Checking this out since my local is broken. I might try to get it working in place instead of cloning fresh. |
You might have to remove those local.settings and |
I skipped this part b/c it seems tailored to single-site:
but I'm already up and running |
Should we be ignoring everything in the .ddev directory except certain files? I'm seeing changes and untracked files there. Unclear from the docs here: https://ddev.readthedocs.io/en/stable/users/topics/whats_in_ddev_dir/ |
I was seeing this when running a
Seeing how quickly I can get #4470 working... :) |
do you not have a .ddev/.gitignore file? |
I do now. Swear that wasn't there before... This PR is leaning towards "just working" as opposed to #4470. Curious if @e-marie-w might have time to try. |
My understanding is that Marie is working through the steps as I type this. Not sure at what point the PR was pulled. Might not have the latest commits |
Our package.json might need updating to handle M1. Seeing all kinds of issues related to node-sass and python for M1s. We are running node:14 due to node-sass (sass/node-sass#3033) and nodejs supports M1 starting at node:15. Temp workaround would be to set the terminal window to use Rosetta nodejs/build#2474 I don't know if it is this simple (when starting a new terminal window): Long term, we need to adopt a higher node version and make sure our build works with the update dependencies. |
We should probably update to Node 16: https://nodejs.org/en/about/releases/ |
Thanks but I forgot to test sitenow_migrate. 🤦🏻 Doing that now... |
Probably want to get everyone currently on this before we merge into main. |
FWIW I had to wipe everything out from testing this to get my vm environment back up and running.. |
Fixed. |
Testing this. |
Am I crazy or is there an extra |
That was added in #4485. We have to specify a no-op for each command we want to override specified in BLT's default |
Got it. I am approving this with words since trying to get to the files page is causing my browser to timeout because of the number of file changes. Nice work! |
Noticed the functional test written in 2595d9f isn't running in CI. Not a blocker, can be fixed in a follow up. |
Converting back to draft until we are ready to merge. Need to update and account for any site provisions/deprovisions, etc. before merge |
Resolves #3641
DDEV UIowa Setup
Note: If you don't have a current setup (M1), you will have to follow some onboarding docs to get git and homebrew setup.
vagrant halt
on your current setupddev
ormain
if this has been merged (hello future)ddev stop --unlist uiowa
Additional Notes
This does not clean up the drupalvm stuff completely.039c7d4ddev mutagen sync
if your containers seem out of sync.To Test
Try to perform normal local environment tasks:
blt frontend
blt ds --site=sandbox.uiowa.edu
drush @sandbox.local cr
Additional resources:
https://iowaweb.slack.com/archives/C8SM82ZFZ/p1638808178159000