- last update
over 1 year ago 43 pass - Issue created by @tsquared212
- First commit to issue fork.
- last update
over 1 year ago 43 pass - @hmdnawaz opened merge request.
- Status changed to Needs work
over 1 year ago 9:51am 24 May 2023 - ๐จ๐ญSwitzerland bircher ๐จ๐ฟ
Hi Thanks for opening an issue when you see something that seems like a problem.
The code you seem not to agree with is
drush_log($text);
.
This function comes from Drush 8. Drush 8 is no longer recommended and is not compatible with Drupal 10.
Config Split 1.x now requires Drupal 9, which in turn is not supported by Drush 8. So instead of trying to "fix" this. We should instead totally remove it! The code path which offends you is never loaded in Drupal 10 since it is Drush 8 code. I was about to close this issue as won't fix. But instead I am retitling it.See https://www.drush.org/11.x/install/#drupal-compatibility
- ๐ฎ๐ณIndia prashant.c Dharamshala
Prashant.c โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia prashant.c Dharamshala
Prashant.c โ changed the visibility of the branch 3362191-remove-drush-8-compatibility to hidden.
- ๐ฎ๐ณIndia prashant.c Dharamshala
Prashant.c โ changed the visibility of the branch 3362191-drupal-10-compatibility to active.
- last update
11 months ago Fetch Error - Status changed to Needs review
11 months ago 10:15am 27 December 2023 - ๐ฎ๐ณIndia prashant.c Dharamshala
As per the comments in https://www.drupal.org/project/config_split/issues/3362191#comment-15074998 ๐ Remove Drush 8 integration Needs review , removing the
config_split.drush.inc
file. Changing status to NR.Thanks