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

Various D8 updates #496

Merged
merged 17 commits into from
Jan 29, 2020
Merged

Various D8 updates #496

merged 17 commits into from
Jan 29, 2020

Conversation

totten
Copy link
Member

@totten totten commented Jan 15, 2020

This gives a set of several Drupal 8-based build-types:

Name Drupal 8 Deployment Style CiviCRM Deployment Style
d8prj-empty drupal-composer/drupal-project None
d8prj-clean drupal-composer/drupal-project civicrm/civicrm-asset-plugin; stable or git
d8prj-re drupal-composer/drupal-project roundearth/civicrm-composer-plugin; stable-only
drupal8-empty D8 Tarball/Drush None
drupal8-clean D8 Tarball/Drush civicrm/civicrm-asset-plugin; stable or git

Each build-type creates the admin and demo users and assigns sufficient permissions to login and navigate to the CiviCRM dashboard.

I've also had some success in getting tests to run:

civibuild create mysite --type drupal8-clean --civi-ver master
civi-test-run -b mysite -j /tmp/junit phpunit-api4 phpunit-e2e

Specific tests do raise issues, but the test suites on the whole are running.

See also: civicrm/civicrm-core#16328 (comment)

@totten totten force-pushed the master-d8-2020 branch 4 times, most recently from b38656d to 988a87e Compare January 16, 2020 03:50
@MegaphoneJon
Copy link
Contributor

cat-cats-kitten-kitty-pic-picture-funny-lolcat-cute-fun-lovely-photo-images-i-love-this-thread-so-much

totten added 16 commits January 27, 2020 18:45
The `civicrm_webtest` module isn't workable on D8.
This build type is useful for reproducing the RoundEarth structure.
However, it has the same limit that it can only load stable releases.
Consequeuntly, the `--civi-ver` option will be required.
…rom totten/5.21-packages

This includes additional fixes for supporting the alternate 'packages', running 'GenCode', etc
The D7 install routine sets the file_private_path via drush.  However, this
was skipped in the first pass at `drupal8_install()`... probably b/c D8
doesn't allow this. D8 requires editing the `settings.php`.

We achieve the same end by injecting code into D8's `settings.php`.
This was enabled in `drupal-{clean,demo,case}`, so do the same.
@totten totten changed the title (WIP) Various D8 updates Various D8 updates Jan 29, 2020
@totten totten merged commit 5cb31a5 into civicrm:master Jan 29, 2020
@totten
Copy link
Member Author

totten commented Jan 29, 2020

Filled in a description; removed "WIP" label; merged.

@totten totten deleted the master-d8-2020 branch January 29, 2020 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants