Skip to content

update README for 5.1 release#1137

Merged
gaoxueyu merged 1 commit intoIvorySQL:masterfrom
jiaoshuntian:update_readme
Dec 18, 2025
Merged

update README for 5.1 release#1137
gaoxueyu merged 1 commit intoIvorySQL:masterfrom
jiaoshuntian:update_readme

Conversation

@jiaoshuntian
Copy link
Copy Markdown
Collaborator

@jiaoshuntian jiaoshuntian commented Dec 18, 2025

Summary by CodeRabbit

  • Documentation
    • Updated installation guides across English and Chinese documentation to reference v5.1.
    • All quick-start links and installation method paths (Yum, Docker, RPM, Source) now point to v5.1 documentation.
    • Expanded developer formatting tools documentation with details on in-tree tool usage and automated workflow checks.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 18, 2025

Walkthrough

Documentation references updated from version v5.0 to v5.1 across README files in both English and Chinese. Installation links, Quick Start references, and developer formatting documentation (pgindent/pg_bsd_indent tools and Cirrus FormatCheck workflow) have been refreshed to reflect the new version.

Changes

Cohort / File(s) Summary
Version reference updates
README.md, README_CN.md
Updated all installation-related links, Quick Start references, and documentation anchors from v5.0 to v5.1 across both English and Chinese README files
Developer tools documentation
README.md
Expanded Developer Formatting hooks section with details on in-tree tool usage (pgindent/pg_bsd_indent) and Cirrus FormatCheck workflow

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Documentation-only changes with straightforward version reference replacements
  • Consistent, repetitive pattern applied uniformly across both files
  • No logic, configuration, or code modifications

Possibly related PRs

  • Add readme_cn #762: Modifies README files including README_CN.md and updates installation links/sections with version references
  • update readme for 4.5 release #813: Updates documentation and installation links in README.md and README_CN.md with version reference changes
  • Update readme #978: Modifies README files to update installation documentation version links and developer formatting/Cirrus workflow documentation

Poem

🐰 From v5.0 we hop to v5.1,
Links and docs refreshed, the work is done!
Chinese, English, all paths aligned,
Tools and workflows—so well-designed!
A documentation update, clean and bright! 📝✨

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 'update README for 5.1 release' directly and accurately summarizes the main changes: updating documentation files (README.md and README_CN.md) with version 5.1 references and links.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3dfb676 and 26d903e.

📒 Files selected for processing (2)
  • README.md (1 hunks)
  • README_CN.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: pg_regression (ubuntu-latest)
  • GitHub Check: contrib_regression
  • GitHub Check: oracle_pg_regression (ubuntu-latest)
  • GitHub Check: build (ubuntu-latest)
  • GitHub Check: oracle_regression (ubuntu-latest)
  • GitHub Check: meson_build (ubuntu-latest)
🔇 Additional comments (2)
README_CN.md (1)

30-30: Verify Chinese documentation URLs match English version structure.

The Chinese README has been updated to reference v5.1 with /cn/ locale paths and Chinese anchors. Ensure these resolve correctly alongside the English documentation and that anchor naming conventions (#yum源安装, #docker安装, #rpm安装, #源码安装) match the actual documentation structure.

The changes mirror the English version appropriately, with correct locale and language-specific anchor naming. Pending verification of the underlying documentation availability, the structural consistency between README.md and README_CN.md is sound.

Also applies to: 32-36

README.md (1)

34-37: Original review comment verified as accurate. The documented tool paths (src/tools/pgindent and src/tools/pg_bsd_indent) and CI workflow description align with standard PostgreSQL development practices. No corrections needed.

Comment thread README.md
Comment on lines +26 to +32
We recommend following our [Quick Start](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/3.1#quick-installation) for how to install and running IvorySQL.

Furthermore, for more detailed installation instructions, please refer to the [Installation Docs](https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/4.1#introduction). We provide four installation methods for IvorySQL, as outlined below:
- [Yum installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/4.1#Yum-installation)
- [Docker installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/4.1#Docker-installation)
- [Rpm installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/4.1#Rpm-installation)
- [Source code installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/4.1#Source-code-installation)
Furthermore, for more detailed installation instructions, please refer to the [Installation Docs](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/4.1#introduction). We provide four installation methods for IvorySQL, as outlined below:
- [Yum installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/4.1#Yum-installation)
- [Docker installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/4.1#Docker-installation)
- [Rpm installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/4.1#Rpm-installation)
- [Source code installation](https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/4.1#Source-code-installation)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Update documentation links to v5.0 — v5.1 does not exist.

IvorySQL 5.0 was released recently as their latest version, not v5.1. All links in lines 26-32 point to non-existent v5.1 documentation. Update them to reference v5.0 instead (e.g., https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/3.1#quick-installation).

🤖 Prompt for AI Agents
In README.md around lines 26 to 32, all documentation URLs point to a
non-existent v5.1 path; update each link to use v5.0 instead (for example change
https://docs.ivorysql.org/en/ivorysql-doc/v5.1/v5.1/3.1#quick-installation to
https://docs.ivorysql.org/en/ivorysql-doc/v5.0/v5.0/3.1#quick-installation) and
similarly replace v5.1 with v5.0 in the Installation Docs and the four specific
installation method links so they point to the correct v5.0 documentation pages.

@gaoxueyu gaoxueyu merged commit 7bcdd6f into IvorySQL:master Dec 18, 2025
6 checks passed
@jiaoshuntian jiaoshuntian deleted the update_readme branch January 5, 2026 09:13
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.

2 participants