Skip to content

docs: correct pg_dump connection environment variables - #343

Merged
gaoxueyu merged 1 commit into
IvorySQL:v5.6from
Wang1rrr:docs/pg-dump-connection-env
Oct 21, 2026
Merged

gaoxueyu merged 1 commit into
IvorySQL:v5.6from
Wang1rrr:docs/pg-dump-connection-env

Conversation

@Wang1rrr

Copy link
Copy Markdown
Contributor

Summary

Fixes #342.

Correct HOST and PORT to PGHOST and PGPORT in the English and Chinese pg_dump connection paragraphs. These are the environment variables mapped by IvorySQL's libpq connection option table.

Only the variable names change; the two translations remain aligned.

Verification

  • Compared the documented names with src/interfaces/libpq/fe-connect.c.
  • git diff --check passed.
  • No database execution or documentation site build was run for this text-only correction.

Use the PGHOST and PGPORT names recognized by libpq in both the English
and Chinese backup instructions.

Fixes IvorySQL#342
Copilot AI balanced review requested due to automatic review settings October 19, 2026 00:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions
github-actions Bot temporarily deployed to pull request October 19, 2026 00:26 Inactive
@github-actions

Copy link
Copy Markdown

🚀 IvorySQL-Docs Preview Ready

@jiaoshuntian

Copy link
Copy Markdown
Contributor

This looks good to merge.
Could you also submit a separate PR with the same correction targeting master, so both maintained branches stay aligned?

@gaoxueyu
gaoxueyu merged commit 80291a4 into IvorySQL:v5.6 Oct 21, 2026
2 checks passed

This branch was previously deployed

1 inactive deployment
pull request — 2e12bb6b Deployed Oct 19, 2026 by github-actions[bot]
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.

docs: pg_dump connection variables are documented as HOST and PORT

4 participants