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

Refactored site install script to rely on drush:deploy. #964

Merged
merged 5 commits into from
Aug 27, 2023

Conversation

AlexSkrypnyk
Copy link
Member

@AlexSkrypnyk AlexSkrypnyk commented Aug 24, 2023

  • Drush 12
  • Downgraded config_split to version 1
  • Refactored site install to use drush:deploy (when config is provided)
  • Introduced DREVOPS_DRUPAL_INSTALL_ENVIRONMENT variable passed to the custom scripts.
  • Added passing of DREVOPS_DRUPAL_INSTALL_OVERRIDE_EXISTING_DB to the custom scripts if the DB was refreshed.
  • Changed custom scripts to contain 2-digit suffix.
  • Refactored site-install tests to use steps_runner
  • Added BATs steps_runner + tests.
  • Improved tests to not contain unbound variables.

closes #953

@AlexSkrypnyk AlexSkrypnyk force-pushed the feature/refactor-site-install branch 3 times, most recently from 58e8f8b to f317c99 Compare August 27, 2023 01:46
@AlexSkrypnyk AlexSkrypnyk merged commit 516555c into main Aug 27, 2023
3 checks passed
@AlexSkrypnyk AlexSkrypnyk deleted the feature/refactor-site-install branch August 27, 2023 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Set DREVOPS_DRUPAL_INSTALL_OVERRIDE_EXISTING_DB=1 when the site is being installed from scratch
1 participant