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.

Sign up for GitHub

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

Jump to bottom

Add initial CI #28

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add initial CI #28

wants to merge 5 commits into from

Conversation

jgaskins
Copy link
Owner

We need to use redis-stack-server so that we can test the Redis modules in use:

  • RediSearch
  • RedisJSON
  • RedisGraph
  • RedisTimeSeries

Adding support for cluster specs will likely be more difficult and require adding a config matrix

Fixes #23

@jgaskins
Add initial CI
23f89c3
@jgaskins
Detach Redis server when running
0818453
@jgaskins
Expose Redis port
5ba09b2
@jgaskins
Try the "services" option
da1475a 
I don't know why I didn't try this before 🙃
@jgaskins
Default connection to REDIS_URL
4c535f6 
This is also what Cluster does, except it uses REDIS_CLUSTER_URL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add CI
1 participant
@jgaskins