Skip to content

Conversation

heckj
Copy link
Collaborator

@heckj heckj commented Jan 17, 2025

follow-on to #3580 - explaining the bits of adding redis.

Notably - that it's NOT required to run the site locally, but available if you want to set that up or work on the relevant code paths that use it.

@heckj heckj self-assigned this Jan 17, 2025
@cla-bot cla-bot bot added the cla-signed label Jan 17, 2025
Copy link
Member

@daveverwer daveverwer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Joe!

@daveverwer daveverwer enabled auto-merge January 17, 2025 22:21
@daveverwer
Copy link
Member

Apologies, @heckj I hadn't spotted that this wasn't on our repo causing the tests not to run. Could you rebase it and I'll get the tests running and let it merge.

Thanks!

@heckj heckj closed this Jan 20, 2025
auto-merge was automatically disabled January 20, 2025 19:35

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants