Problem/Motivation

README.md and example.services.yml files use a path that will cause an error with composer based installs.
I know that these paths are provided just as examples, but we can update them so they work out of the box.

Old path: modules/redis/example.services.yml
New path: modules/contrib/redis/example.services.yml

Proposed resolution

Update the path to match the module location in composer based installs.

This change is pretty harmless and nothing else will be affected.

CommentFileSizeAuthor
#2 redis-container-paths-3196686-1.patch1.07 KBaramboyajyan

Comments

aramboyajyan created an issue. See original summary.

aramboyajyan’s picture

aramboyajyan commented
StatusFileSize
new redis-container-paths-3196686-1.patch1.07 KB
milovan’s picture

milovan commented
Status: Needs review » Reviewed & tested by the community

Yes, you are right. +1 and it works

freddy rodriguez’s picture

freddy rodriguez
Location Bogotá
commented

Thanks for this! +1

  • Berdir committed 51c5417 on 8.x-1.x 
    Issue #3196686 by aramboyajyan: Update container paths in README and...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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