-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: Remove the loop condition in the test code to fix CI #3
Conversation
[NEW] Added WatcherConstant.java [NEW] Added LettuceRedisWatcher.java [NEW] Added LettuceSubscriber.java [NEW] Added LettuceSubThread.java [NEW] Added LettuceRedisWatcherTest.java [UPDATE] Updated .gitignore [NEW] Added .releaserc.json [NEW] Added maven-settings.xml [NEW] Added pom.xml [UPDATE] Updated README.md
… of remote IP address for testing.
…n and update logic.
@tangyang9464 @imp2002 please review |
@ShingmoYeung plz fix conflicts |
@hsluoyz permission denied? |
@ShingmoYeung it's your PR. You of course have rights to modify it. If you cannot do it by web, you can modify your commits in git |
# Conflicts: # src/test/java/org/casbin/test/LettuceRedisWatcherTest.java
…uster. - Add Redis URI constants. - Update Redis connection initialization in LettuceRedisWatcher. - Add constructors for different Redis connection types. - Add support for Redis Cluster in LettuceRedisWatcher. - Update LettuceRedisWatcherTest to test Redis Cluster connection.
…ver instead of the specific IP address.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #3 +/- ##
=========================================
Coverage ? 82.08%
=========================================
Files ? 4
Lines ? 134
Branches ? 11
=========================================
Hits ? 110
Misses ? 17
Partials ? 7 ☔ View full report in Codecov by Sentry. |
@hsluoyz fixed it, code-review please |
@ShingmoYeung plz fix: |
🎉 This PR is included in version 1.0.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Remove the loop condition in the test code.
Fix: #2