release: v0.6.0#125
Conversation
WalkthroughVersion bump to 0.6.0 across SDK and project metadata, plus changelog updates introducing an Unreleased section and a new v0.6.0 release entry. Public constant Version and DefaultUserAgent updated; project file Version property updated. No code logic or control-flow changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (31.69%) 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 #125 +/- ##
=======================================
Coverage 31.69% 31.69%
=======================================
Files 135 135
Lines 6862 6862
Branches 900 900
=======================================
Hits 2175 2175
Misses 4499 4499
Partials 188 188 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/OpenFga.Sdk/Configuration/Configuration.cs (1)
100-112: Bug: BasePath fallback uses ApiUrl in the ApiHost check (likely typo).The second branch reads
if (ApiHost != null && ApiUrl != ""), which prevents the deprecated ApiHost/Scheme fallback from ever triggering when ApiUrl is empty. Should checkApiHostinstead ofApiUrl.Apply:
- if (ApiHost != null && ApiUrl != "") { + if (!string.IsNullOrEmpty(ApiHost)) { return $"{ApiScheme ?? "https"}://{ApiHost}"; }
🧹 Nitpick comments (1)
CHANGELOG.md (1)
19-21: Fix GitHub admonition formatting for the breaking changes.Bare
[!WARNING]won’t render as an admonition on GitHub without a blockquote. Either remove it (you already have “BREAKING CHANGES:”) or format properly.-[!WARNING] -BREAKING CHANGES: +> [!WARNING] +> BREAKING CHANGES:
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
CHANGELOG.md(1 hunks)src/OpenFga.Sdk/Configuration/Configuration.cs(1 hunks)src/OpenFga.Sdk/OpenFga.Sdk.csproj(1 hunks)
🔇 Additional comments (3)
src/OpenFga.Sdk/Configuration/Configuration.cs (1)
76-79: Version and default User-Agent updated to 0.6.0 — good.Aligned with the package version; no functional risk.
CHANGELOG.md (1)
3-7: Unreleased anchor and v0.6.0 section look correct.Links and date align with the release.
src/OpenFga.Sdk/OpenFga.Sdk.csproj (1)
15-15: Verify no stale ‘0.5.1’ references remainVersion bump aligns with Configuration.Version and CHANGELOG. Manually search across the repo—for example:
grep -R --exclude-dir=bin --exclude-dir=obj -n '0\.5\.1' .
Description
Note: This is being rolled out first so that the v0.7.0 can focus on the .NET Standard 2.0 support, allowing users who face issues with that release to still be able to benefit from these other fixes.
Changes:
start_timeparameter inReadChangesendpointExpandListStores3from15(fix : change default retry policy sdk-generator#420) - thanks @ovindu-aListRelationsshould not swallow errors[!WARNING]
BREAKING CHANGES:
ListStoresmethod now accepts a body parameter with an optionalNameto filter the stores. This is a breaking change as it changes the method contract to allow passing in a body with the name.What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit
Documentation
Chores