-
Notifications
You must be signed in to change notification settings - Fork 10
DDEV 1.23, sqlite, yarn. #30
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
Conversation
Feels to me that there are some missing instruction somewhere, with the changes in this PR, running a Functional test:
If I add
|
@simesy there are massive differences between this PR and the one you previously proposed (#23)
Difference between the two PRs
-# Clone the Drupal core codebase into the current directory. -# Configure and start a minimal DDEV project. -# Use DDEV to install Drupal's composer dependencies. The host machine does not need PHP. -# Get the latest release of this plugin. This adds files to the .ddev folder.
-# This plugin provides a "drupal" command, see what it offers. -# Install (or re-install) Drupal. -# Run a small sample of PHPUnit tests. -# Run Nightwatch tests. -# Todo, I have temporarily removed a lot of examples here.
By closing the other one, you invalidated all the reviews there. And then opening this one with major changes leaves us all confused. Would you mind reopening the other one and closing this one? If you need help figuring out what was going wrong, I'm happy to help. If you'd rather, I can open a replacement PR for #23 I guess and give you credit in the commit. |
Per conversation with @simesy this can be closed in favor of |
* DDEV 1.23 compatiblity, Drupal core yarn/sqlite compatibility. Docs. * Need to use sqlite now b/c no mysql. From /pull/27/files. * Remove install of sqlite3 and make dependent on DDEV v1.23.1 * Run tests now that sqlite 3.45 in DDEV release * Dockerfile is gone, don't try to install it * add back in omit_containers, can remove farther down the line * Remove --verbose from phpunit command, not supported in phpunit 10 --------- Co-authored-by: Simon Hobbs <[email protected]>
Updates the plugin for ddev 1.23.
The original issue was here, however the base branch changed so i need to create a new PR.
Test by cloning the forked repo and using slightly different commands.