Skip to content

Conversation

Oscmage
Copy link

@Oscmage Oscmage commented Sep 27, 2025

Description

What problem is being solved?

Updating the documentation to Postgres 18

References

Depends on PR https://github.com/openfga/openfga/pull/2722/files

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

  • Documentation
    • Updated Docker setup instructions to use PostgreSQL 18 for the database service.
    • Ensures users pulling the latest images have a consistent, up-to-date environment when running via Docker Compose.
    • No changes to application behavior; guidance only.
    • If you rely on a specific PostgreSQL version, review your local setup before upgrading to avoid unexpected differences in development environments.

Copy link
Contributor

coderabbitai bot commented Sep 27, 2025

Walkthrough

Updated a documentation page to change the Docker Compose Postgres image tag from postgres:17 to postgres:18 in the OpenFGA setup guide. No other files or logic were modified.

Changes

Cohort / File(s) Summary of Changes
Docs: Docker setup
docs/content/getting-started/setup-openfga/docker-setup.mdx
Updated Docker Compose snippet to reference postgres:18 instead of postgres:17.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly conveys the primary change of the pull request, namely updating the documentation to use PostgreSQL version 18, and directly reflects the modifications made in the Docker setup guide.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between db0dd24 and a4dd56b.

📒 Files selected for processing (1)
  • docs/content/getting-started/setup-openfga/docker-setup.mdx (1 hunks)

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.

@Oscmage Oscmage marked this pull request as ready for review September 27, 2025 10:06
@Oscmage Oscmage requested a review from a team as a code owner September 27, 2025 10:06
@dyeam0 dyeam0 added the documentation Improvements or additions to documentation label Sep 30, 2025
Copy link
Member

@dyeam0 dyeam0 left a comment

Choose a reason for hiding this comment

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

Thanks for the update @Oscmage!

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants