Skip to content

Conversation

@pjcdawkins
Copy link
Collaborator

@pjcdawkins pjcdawkins commented Aug 6, 2025

Why

Parallel to platformsh-templates/drupal11#11

Simplify Drush configuration removing the need for a ./drush/upsun_generate_drush_yml.php file.

What's changed

  • Move now-unnecessarily complex Drush configuration from a PHP script to a couple of lines in .environment
  • Remove now unnecessary reference to the config-reader library for Upsun + Drupal
  • Remove dependencies which was unnecessarily requiring Composer 2 (it's default in PHP 8.2+)
  • Inline the deploy script

I'm ignoring the nextjs-drupal template and tutorial (we could handle that separately?), and ancient Drupal versions (< 10).

Where are changes

Updates are for:

  • platform (sites/platform templates)
  • upsun (sites/upsun templates)

@github-actions
Copy link

github-actions bot commented Aug 6, 2025

@pjcdawkins pjcdawkins requested a review from flovntp September 15, 2025 16:45
@flovntp flovntp merged commit 5f8396e into main Oct 24, 2025
14 checks passed
@flovntp flovntp deleted the simplify-drupal-11-drush-config branch October 24, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants