Problem/Motivation

The current implementation of the Acquia CMS Image module includes a dependency on the Acquia CMS Common module. So, installing Acquia CMS Image module installs Acquia CMS Common module, which in turn brings in several additional modules that are not necessary for Acquia CMS Image to function. To streamline dependencies, we should remove the reliance on Acquia CMS Common and instead declare only the modules directly required by Acquia CMS Image (e.g., Field Group).

Proposed resolution

  1. Review and limit module dependencies so that only modules essential to Acquia CMS Image remain enabled.
  2. Update the GitLab CI template and use CI template provided by Drupal Core, replacing the current ORCA setup, and remove any files specific to ORCA.
  3. Add / Update code in Acquia CMS Image module for module to function as expected.
  4. Refactor and update PHPUnit tests to ensure they are passing.
  5. Ensure all CI pipelines complete successfully, including those in the "allowed failures" category.
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

chandu7929 created an issue. See original summary.

vishalkhode’s picture

Title: Remove acquia_cms_common module dependency » Decouple Acquia CMS Image from Common Module and Modernize CI Pipeline
Version: 1.6.4 » 1.x-dev
Category: Task » Feature request
Issue summary: View changes

vishalkhode’s picture

Status: Active » Needs review
ankitv18’s picture

Left a comment, please check.
Keeping this in review for other inputs

  • vishalkhode committed 7035aab3 on 2.x
    Issue #3538820 by vishalkhode: Decouple Acquia CMS Image from Common...
vishalkhode’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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