- Issue created by @parkh
- 🇦🇺Australia silverham
Attaching patch uses static variable instead, so format is not locked in. not sure why it added twice though.
- Status changed to Needs review
9 months ago 12:58pm 26 July 2024 - Status changed to Needs work
6 months ago 5:30pm 18 October 2024 - 🇺🇸United States andyg5000 North Carolina, USA
The init hook is getting registered twice by Drush because of a drush.services.yml and the auto-discovery service that Drush invokes. We should remove the command in drush.services.yml and let the auto-discovery handle registering the commands.
I would like the module maintainers to review this before making an MR because there are several commands that are impacted by this.
- 🇺🇸United States andyg5000 North Carolina, USA
Here's a temp patch I'm using for now until we can get the module maintainers eyes on this. I don't think we need to reference any of the drush.services.yml stuff and removing it fixes the duplicate hook registrations.
- Status changed to RTBC
17 days ago 10:23am 25 March 2025 - 🇧🇪Belgium lisotton Brussels
I'm also facing this issue and it took me hours to realize that the issue was the auto-discovery + the drush.services.yml.
@andyg5000 your patch #9 is working fine, so why not moving this ticket to RTBC?
Maybe we can speed up and catch more attention from the maintainers if you also open a MR in Gitlab.