Overview

:pray: Please eliminate in favor of auto-wiring. See `canvas.services.yml` for an example :blush:

— yours truly at https://git.drupalcode.org/project/canvas/-/merge_requests/646#note_727539

Proposed resolution

Automate this feedback.

User interface changes

None.

Issue fork canvas-3581143

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:

About issue forks

Comments

wim leers created an issue. See original summary.

wim leers opened merge request !798

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: wim leers » Unassigned
Issue tags: +AI-accelerated, +Novice, +Chicago2026

Ephemeral d.o GitLab issues apparently. MR at https://git.drupalcode.org/project/canvas/-/merge_requests/798

This will surface 8 errors.

That makes for a great Novice task, great DrupalCon timing 😄

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

the_turk’s picture

the_turk commented

Updated MR to enforce autowiring and disallow explicit arguments in services.yml.

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

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

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs work

Rebased, but looks like it needs a little work for an errant service argument and test failures.