- Issue created by @torfj
- Merge request !2Remove unnecessary Group module dependency from base module → (Merged) created by torfj
- 🇺🇸United States torfj Seattle, WA
Hi maintainers, I made a MR with the suggested changes. Let me know what you think.
When installing the Content Insights Report module using Composer, the drupal/group module and its dependencies (entity and flexible_permissions) are always installed even if you're only using the base module functionality without the content_insights_report_group submodule.
The main module's content_insights_report.info.yml only depends on drupal:node, but the composer.json requires drupal/group for both the main package and development, causing unnecessary modules to be installed on sites that don't use the group functionality.
Also, I noticed that the main module requires drupal/group:^2.3, but the submodule content_insights_report_group requires drupal/group:^3.
- Move the Group dependency from the main module's composer.json to the "suggest" section.
- Ensure consistent versioning between the main module and submodule.
Active
1.0
Code
Hi maintainers, I made a MR with the suggested changes. Let me know what you think.