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
GEODE-10278: Remove the geode-for-redis module #7657
GEODE-10278: Remove the geode-for-redis module #7657
Conversation
Remove the geode-for-redis module. This commit is just the module itself, not the related documentation and CI scripts.
Removing geode-for-redis from all of the places that reference it - CI, scripts, docs, expected assembly and pom context.
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.
I have not looked through all the files that are changed. As this PR is to remove the redis module; assuming it does not impact the other functionalities of the product (Greep PR pipelines); approving these changes.
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.
+1 for the docs portion of the PR.
Good, thorough job of removing the related sources and snippets.
Remove the geode-for-redis module and related documentation.
To make this easier to review, I split out the change that just deletes the geode-for-redis directory into a separate commit.
For all changes:
Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
Has your PR been rebased against the latest commit within the target branch (typically
develop
)?Is your initial contribution a single, squashed commit?
Does
gradlew build
run cleanly?Have you written or updated unit tests to verify your changes?
If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?