Problem/Motivation
The current implementation of the Acquia CMS Audio module includes a dependency on the Acquia CMS Common module. So, installing Acquia CMS Audio module installs Acquia CMS Common module, which in turn brings in several additional modules that are not necessary for Acquia CMS Audio 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 Audio (e.g., Field Group).
Proposed resolution
- Review and limit module dependencies so that only modules essential to Acquia CMS Audio remain enabled.
- 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.
- Add / Update code in Acquia CMS Audio module for module to function as expected.
- Refactor and update PHPUnit tests to ensure they are passing.
- Ensure all CI pipelines complete successfully, including those in the “allowed failures” category.
Issue fork acquia_cms_audio-3539119
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
Comment #2
vishalkhode commentedComment #4
vishalkhode commentedComment #5
ankitv18 commentedLeft a comment, please check.
Keeping this in review for other inputs
Comment #7
vishalkhode commented@ankitv18: Backward compatibility is already taken care of. Please check acquia_cms_audio.module and acquia_cms_audio.install file.