fix: MCD-610: Update gitHub Actions and fix Drupal 11 compatibility #76
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
petarbasic
commented
Jan 15, 2026
petarbasic
commented
- Update actions/cache from v2.1.5 to v4
- Replace deprecated set-output with $GITHUB_OUTPUT
petarbasic
force-pushed
the
feature/MCD-610
branch
from
d1b9bef to
dbae38c
Compare
- Update actions/cache from v2.1.5 to v4 - Replace deprecated set-output with $GITHUB_OUTPUT Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
petarbasic
force-pushed
the
feature/MCD-610
branch
from
dbae38c to
747a28f
Compare
The old version 22.2.0 no longer exists on Docker Hub. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The solr-8-drupal image doesn't have 25.12.0, only up to 25.9.0. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Newer Docker versions try to pull CLI_IMAGE from Docker Hub before building it locally. Building cli first ensures it's available. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Temporarily use feature/MCD-610 branch of devsetup-docker which has the fix for deprecated npm options (unsafe-perm removed in npm 9+). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Drupal 10 is end of life. Update to Drupal 11: - Update drupal/core-* packages from ^10 to ^11 - Update symfony/dotenv from ^5.4 to ^7.0 (Drupal 11 uses Symfony 7) - Update PHP platform version from 8.1 to 8.3 - Update GitHub Actions PHP version to 8.3 - Update Docker PHP version to 8.3 - Update version check from "Drupal 10" to "Drupal 11" Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace deprecated assertion handler with ini_set for Drupal 11:
- Remove assert_options(ASSERT_ACTIVE, TRUE)
- Remove \Drupal\Component\Assertion\Handle::register()
- Add ini_set('zend.assertions', 1)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Update README to reflect Drupal 11 requirement with PHP 8.3+. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fago
approved these changes
Jan 20, 2026
fago
changed the title
fago
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.