Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No include of the package/library "acquia/memcache" if not required #748

Open
joshirohit100 opened this issue Dec 26, 2024 · 1 comment
Open
Labels
bug Something isn't working

Comments

@joshirohit100
Copy link
Contributor

joshirohit100 commented Dec 26, 2024

When a new project/app is started, this template by default includes the acquia/memcache https://github.com/acquia/drupal-recommended-project/blob/master/composer.json#L17

While if project is started on Acquia Cloud Next, then this package is not required. Ref - https://github.com/acquia/memcache-settings?tab=readme-ov-file#acquia-memcache-settings

So, I think this should not be included by default (as then manage and upgrade etc) but may be some documentation to include this if need and not Acquia Cloud Next Project.

(Created this as bug though not causing any functional/app issue)

@joshirohit100 joshirohit100 added the bug Something isn't working label Dec 26, 2024
@vishalkhode1
Copy link
Contributor

We've already added instruction in README to remove the acquia/memcache, if plan is to deploy project on ACN. So, this probably will be closed, unless team thinks otherwise.
cc: @vipin-mittal-acquia / @nagwani

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants