Conversation
WalkthroughA version release from 0.9.6 to 0.9.7, updating version numbers across Gradle build files, example projects, changelog, README, and SDK constants. Changelog documents new StreamingApiExecutor feature and bug fixes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Related Documentation 1 document(s) may need updating based on files changed in this PR: OpenFGA's Space Retry-After Header and Retry StrategyView Suggested Changes@@ -70,7 +70,7 @@
- For details on supported Node.js versions and our support policy, see [SUPPORTED_RUNTIMES.md](https://github.com/openfga/js-sdk/blob/main/SUPPORTED_RUNTIMES.md).
### Java SDK
-**Version Introduced:** v0.9.6
+**Version Introduced:** v0.9.7
**[Changelog](https://github.com/openfga/java-sdk/blob/33a85203ec0af0935b0fa265a469a1851870fff7/CHANGELOG.md#L40-L52)**
#### Retry-After Handling
@@ -209,7 +209,7 @@
| SDK | Retry-After Support | Exponential Backoff (Default) | Max Retries (Default/Max) | Min Retry Delay (Default) | Error Exposure | Version Introduced |
|-------------|--------------------|-------------------------------|---------------------------|--------------------------|---------------|-------------------|
| JavaScript | Yes (int/date, 1–1800s) | 2^n * minWaitInMs, jitter, cap 120s | 3 / 15 | 100ms | Error classes | v0.9.1 |
-| Java | Yes (int/date, 1–1800s) | 2^n * minDelay, jitter, cap 120s | 3 / 15 | 100ms | Error + header | v0.9.6 |
+| Java | Yes (int/date, 1–1800s) | 2^n * minDelay, jitter, cap 120s | 3 / 15 | 100ms | Error + header | v0.9.7 |
| Go | Yes (int/date, 1–1800s) | 2^n * minWaitInMs, jitter, cap 120s| 3 / 15 | 100ms | Error methods | v0.7.0 |
| Python | Yes (int/date, 1–1800s) | 2^n * min_wait_in_ms, jitter, cap 120s | 3 / 15 | 100ms | Exception | v0.9.4 |
| .NET | Yes (int/date, 1–1800s) | 2^n * 500ms, jitter, cap 120s | 3 / 15 | 100ms | Error + header | v0.8.0 |Note: You must be authenticated to accept/decline updates. |
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (38.03%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #307 +/- ##
=========================================
Coverage 38.03% 38.03%
Complexity 1258 1258
=========================================
Files 198 198
Lines 7643 7643
Branches 885 885
=========================================
Hits 2907 2907
Misses 4598 4598
Partials 138 138 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
build.gradle (1)
22-22: Consider centralizing release version from a single source of truth.Line 22 is correct, but this PR shows the version duplicated across several Gradle files. Consider sourcing all version references from one shared property (for example
gradle.properties) to reduce future release drift.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@build.gradle` at line 22, The project currently hardcodes version = '0.9.7' in build.gradle; centralize this by adding a single property (for example version=0.9.7) to gradle.properties and change build.gradle to read that property instead of the literal (replace the direct assignment to the version variable with a property lookup using the Gradle project/property API). Update any other Gradle files that currently duplicate the literal to reference the same property so all modules use the single source of truth (gradle.properties -> project.version or property("version")).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@build.gradle`:
- Line 22: The project currently hardcodes version = '0.9.7' in build.gradle;
centralize this by adding a single property (for example version=0.9.7) to
gradle.properties and change build.gradle to read that property instead of the
literal (replace the direct assignment to the version variable with a property
lookup using the Gradle project/property API). Update any other Gradle files
that currently duplicate the literal to reference the same property so all
modules use the single source of truth (gradle.properties -> project.version or
property("version")).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 3779307b-cb5a-4514-9d6d-2afc14675b1c
📒 Files selected for processing (8)
CHANGELOG.mdREADME.mdbuild.gradleexamples/basic-examples/build.gradleexamples/opentelemetry/build.gradleexamples/streamed-list-objects/build.gradlepublish.gradlesrc/main/java/dev/openfga/sdk/constants/FgaConstants.java
Description
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit
New Features
Bug Fixes