Problem/Motivation

automatic_updates_requirements() checks for existence of a composer_stager class to ensure the module was required with Composer instead of tarball.

we already have this check in package_manager_requirements()

Proposed resolution

remove this redundant check from automatic_updates_requirements()

Issue fork automatic_updates-3248312

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

tedbow created an issue. See original summary.

tedbow opened merge request !107

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Issue summary: View changes
Parent issue: » #3244939: [Plan] Move more functionality into package_manager sub-module to enable project_browser use case
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Active » Needs review
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Title: Move Composer dependencies check on install to package_manager » Remove redundant install-time Composer dependencies check
Status: Needs review » Reviewed & tested by the community

  • tedbow committed 05cc57c on 8.x-2.x 
    Issue #3248312: Remove redundant install-time Composer dependencies...
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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