Create a 1.17 release [#3327909]

Problem/Motivation

The fix in #3278542: Update $logger definition to be compatible with Drupal 10 is necessary for true Drupal 10 support, and even though it has been committed a few months ago, there is still no release that includes that fix.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comment	File	Size	Author
#2	fix-drush-crash.patch	443 bytes	mindaugasd

Comments

Comment #1

18 December 2022 at 19:43

jcnventura created an issue. See original summary.

Comment #2

mindaugasd commented 25 December 2022 at 11:56

Parent issue:

» #3278542: Update $logger definition to be compatible with Drupal 10

Status	File	Size
new	fix-drush-crash.patch	443 bytes

Sending patch for 1.16 to prevent module from crashing drush while where is no release.

Commit in that issue creates a different crash, so therefor made this simpler patch.

Comment #3

jcnventura

English

commented 27 December 2022 at 14:04

The commit did the proper fix to the issue, but it does remove one service injection, so maybe some caches may need to be cleared. In any case, it would be good if you could reveal which "different crash" got created by applying that change, as it might be useful to fix that before 1.17 is created.

Comment #4

mindaugasd commented 1 January 2023 at 18:59

I tried to recreate the error, and found the problem with the site itself (weird directory structure and duplicated modules).
And this patch I made it by copying it from another module issue queue without looking into details.
So green light from me. Original commit does not seem to create another error.