-
Notifications
You must be signed in to change notification settings - Fork 8
Spring Boot Starter for Spring Data Valkey #35
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
Merged
Merged
Changes from 19 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
cfd7d6f
Add standalone Redis starter from Spring Boot 3.5.1
jeremyprime 1ec5a1f
Replace Gradle with Maven
jeremyprime bf382a2
Rebrand from Redis to Valkey
jeremyprime bb9baa5
Add initial Valkey GLIDE support
jeremyprime 4f95ca1
Add Spring Boot example
jeremyprime 4f04884
Add GLIDE cluster support
jeremyprime 3a4d08d
Add Valkey docker and testcontainer support
jeremyprime 68b682a
Update spring-data-valkey version
jeremyprime 307b995
Add GLIDE connection pooling support
jeremyprime 90ebf88
Add GLIDE health actuator support
jeremyprime 9f41685
Fix actuator runtime dependencies
jeremyprime 4853879
Update migration guide to be Spring Boot centric
jeremyprime 453bfda
Add DataValkeyTest support
jeremyprime e267482
Update READMEs
jeremyprime bbf8934
Add configuration to some examples
jeremyprime cbbc887
Update quickstart example and migration guide
jeremyprime c97318d
Update starter POM for release
jeremyprime eb6ba88
Update CI to include starter
jeremyprime 12df6ee
Merge remote-tracking branch 'upstream/main' into spring-boot
jeremyprime cdb8928
Small fixup in ci workflow
ikolomi ba0e5e9
Small fixup in release workflow
ikolomi 67549a0
Prettier fix
ikolomi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,6 +15,7 @@ build | |
| .vscode/ | ||
| out | ||
| work | ||
| appendonlydir | ||
| *.rdb | ||
| *.aof | ||
| build/ | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
should not the test target come before the install?
Also, why not to split the invocations into the separate steps - i will make life easire in the github UI
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.
Once we make the project multi-module this can be simplified, for now we need to manually install Spring Data Valkey before we can use the starter as it relies on Spring Data Valkey and the two modules are not connected through a parent module yet.