Problem/Motivation

Drupal core 10.1 finally supports autowiring: https://www.drupal.org/node/3323122

This means that our service definitions can use autowiring by default. They will still have their string IDs so as not to break anything, but new services won't need to define their arguments manually, and we can change our existing services to autowire their own dependencies painlessly.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

phenaproxima created an issue. See original summary.

wim leers’s picture

Status: Active » Needs work

Looking great! Just 3 nits, then RTBC.

phenaproxima’s picture

Status: Needs work » Needs review
wim leers’s picture

Status: Needs review » Reviewed & tested by the community

Assuming that this will still work after #3345039: Remove dependency on symfony/config: 🚢

tedbow made their first commit to this issue’s fork.

  • tedbow committed 6394bcca on 3.0.x authored by phenaproxima
    Issue #3345028 by phenaproxima, Wim Leers: Package Manager's services...
tedbow’s picture

Status: Reviewed & tested by the community » Fixed

Wow, this is great! Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.