-
Notifications
You must be signed in to change notification settings - Fork 568
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
Update Redis config source reference #11145
Conversation
Signed-off-by: Emmanuel Ferdman <[email protected]>
@emmanuel-ferdman thanks for this PR. Can you put in the comment what problem it addreses? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating the doc
@dippindots @haynescd please review and merge this small PR when you get the chance 🙌 |
Quality Gate passedIssues Measures |
@emmanuel-ferdman Thanks for updating the documentation, this pr is now merged |
Describe changes proposed in this pull request:
Commit 4c3e035 moved the location of
RedisConfig.java
which leads to broken links in the docs. This PR adjusts sources to changes.Checks
Any screenshots or GIFs?
If this is a new visual feature please add a before/after screenshot or gif
here with e.g. Giphy CAPTURE or Peek
Notify reviewers
Read our Pull request merging
policy. It can help to figure out who worked on the
file before you. Please use
git blame <filename>
to determine thatand notify them either through slack or by assigning them as a reviewer on the PR