Problem/Motivation

There is a new ExtensionInstallCommand, it should be in the extension namespace

Steps to reproduce

Open the files.

Proposed resolution

Move the command to the extension namespace
Move the test to the extension test namespace

Remaining tasks

Review

User interface changes

N/A

Introduced terminology

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

Issue fork drupal-3623094

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

nicxvan created an issue. See original summary.

nicxvan opened merge request !17123

nicxvan’s picture

nicxvan commented
Issue summary: View changes
nicxvan’s picture

nicxvan commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Move LGTM to me, since this is brand new and merged on 9/8 shouldn't be out in the wild to break anything.

  • catch committed b9b3d591 on main 
    task: #3623094 Move new install command to the correct namespace

By:...
catch’s picture

catch
he/him
Primary language English
commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

Yes let's do this before we have to add bc for it. Committed/pushed to main and cherry-picked to 11.x, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • catch committed c228d5c9 on 11.x 
    task: #3623094 Move new install command to the correct namespace

By:...
moshe weitzman’s picture

moshe weitzman commented

FWIW Command classes are marked @internal because they are intended to just be some glue that calls into services.

nicxvan’s picture

nicxvan commented

Yes, we still regularly provide BC for most things that are internal.

Status: Fixed » Closed (fixed)

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