- Issue created by @robertoperuzzo
- Merge request !1Issue #3538398 by robertoperuzzo: Finalize the recipe → (Open) created by robertoperuzzo
- 🇮🇹Italy robertoperuzzo 🇮🇹 Tezze sul Brenta, VI
It looks like we can simplify the recipe’s `composer.json` by removing some redundant module requirements:
- `drupal/key` The [`key` module](https://git.drupalcode.org/project/key) is already a dependency of the `ai` module (see [`ai.info.yml`](https://git.drupalcode.org/project/ai/-/blob/1.2.x/ai.info.yml#L9) and [`composer.json`](https://git.drupalcode.org/project/ai/-/blob/1.2.x/composer.json#L6)).
When you run the commandddev drupal-get-module ai 1.2.x
the `key` module is installed automatically.
Therefore, we can safely remove `drupal/key` from the recipe’s `composer.json`. - `drupal/menu_link_content` The `menu_link_content` module is used internally by the `token` module (for example, see [token.module#L705](https://git.drupalcode.org/project/token/-/blob/8.x-1.x/token.module#L705)). Since `token` is already a `require-dev` dependency of the `ai` module, there’s no need to declare `menu_link_content` in the recipe.
- `drupal/project_browser` The `project_browser` module is listed as a `require-dev` dependency for the `eca` module, which in turn is a `require-dev` dependency for the `ai` module. This means `project_browser` is already pulled in indirectly, so we can remove it from the recipe’s `composer.json`.
- `drupal/key` The [`key` module](https://git.drupalcode.org/project/key) is already a dependency of the `ai` module (see [`ai.info.yml`](https://git.drupalcode.org/project/ai/-/blob/1.2.x/ai.info.yml#L9) and [`composer.json`](https://git.drupalcode.org/project/ai/-/blob/1.2.x/composer.json#L6)).
- 🇮🇹Italy robertoperuzzo 🇮🇹 Tezze sul Brenta, VI
Regarding the amazee.io provider I link here the Slack thread.
- 🇺🇸United States thejimbirch Cape Cod, Massachusetts
The changes you've made so far look great.