Skip to content

chore: add api executor example to release please config#329

Open
SoulPancake wants to merge 2 commits intomainfrom
chore/fix-add-api-executor-example-rp
Open

chore: add api executor example to release please config#329
SoulPancake wants to merge 2 commits intomainfrom
chore/fix-add-api-executor-example-rp

Conversation

@SoulPancake
Copy link
Copy Markdown
Member

@SoulPancake SoulPancake commented Apr 21, 2026

Description

What problem is being solved?

How is it being solved?

What changes are made to solve it?

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Updated build configuration and release automation tooling to enhance the development workflow.

@SoulPancake SoulPancake requested a review from a team as a code owner April 21, 2026 14:11
Copilot AI review requested due to automatic review settings April 21, 2026 14:11
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3c16efe6-2dca-448d-986e-4161ca9aa4a4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

Two configuration files are updated to enable release-please tooling to manage version updates for the api-executor build.gradle file. An inline comment marker is added to the dependency declaration, and the build file is registered as an extra file in the release-please configuration.

Changes

Cohort / File(s) Summary
Release configuration
release-please-config.json
Added examples/api-executor/build.gradle as an extra-file entry for release-please to track version updates.
Build script annotation
examples/api-executor/build.gradle
Added inline comment // x-release-please-version to the SDK JAR dependency implementation line to mark the version location for tooling.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • rhamzeh
  • jimmyjames
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding the api-executor example to release-please configuration in two files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/fix-add-api-executor-example-rp

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@examples/api-executor/build.gradle`:
- Line 26: Replace the hard-coded SDK JAR reference so both examples use the
same version: update the dependency string implementation
files('../../build/libs/openfga-sdk-0.9.6.jar') to match the other example's
openfga-sdk-0.9.7.jar (or vice versa), ensuring the implementation files(...)
entry uses the identical openfga-sdk-<version>.jar across examples to avoid
compatibility mismatches.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2e6ac182-38db-4bd3-8895-14bdc13faa16

📥 Commits

Reviewing files that changed from the base of the PR and between 2565cc8 and a285e05.

📒 Files selected for processing (2)
  • examples/api-executor/build.gradle
  • release-please-config.json

Comment thread examples/api-executor/build.gradle Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates release-please configuration so the new api-executor Gradle example participates in automated version bumping, and annotates the example’s SDK jar reference for release-please.

Changes:

  • Add examples/api-executor/build.gradle to release-please-config.json extra-files.
  • Add // x-release-please-version marker to the api-executor example’s local SDK jar dependency line.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
release-please-config.json Includes the api-executor example Gradle file in release-please’s version update targets.
examples/api-executor/build.gradle Adds release-please version marker to the local SDK jar dependency reference.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread examples/api-executor/build.gradle Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.04%. Comparing base (2565cc8) to head (8d919c1).

❌ Your project status has failed because the head coverage (38.04%) 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     #329      +/-   ##
============================================
- Coverage     38.05%   38.04%   -0.02%     
+ Complexity     1259     1258       -1     
============================================
  Files           198      198              
  Lines          7646     7646              
  Branches        885      885              
============================================
- Hits           2910     2909       -1     
  Misses         4598     4598              
- Partials        138      139       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants