Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc_ja_15からのバックポート #2622

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

noborus
Copy link
Contributor

@noborus noborus commented May 18, 2023

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。
@noborus noborus added the version 14 version 14を対象 label May 18, 2023
@KenichiroTanaka
Copy link
Contributor

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。
があるので
#2617
の結論を待ちます。

@noborus
Copy link
Contributor Author

noborus commented May 25, 2023

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。
があるので
#2617
の結論を待ちます。

それは関係ないと思います。

@KenichiroTanaka
Copy link
Contributor

KenichiroTanaka commented May 30, 2023

それは関係ないと思います。
「本来省かずにつけておく用語が省かれていたものの修正」の方ですね。

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。

#2605
のバックポートにさらに修正を含むという理解で合っているでしょうか?

気にしているのはレビュー範囲(どこを見れば良いかの範囲)とdoc_15との整合性です。

@noborus
Copy link
Contributor Author

noborus commented May 30, 2023

それは関係ないと思います。
「本来省かずにつけておく用語が省かれていたものの修正」の方ですね。

さらに、オーバ[^ー]、ワーカ[^ー]、スーパ[^ー]を統一するために
追加で修正しています。

#2605
のバックポートにさらに修正を含むという理解で合っているでしょうか?

気にしているのはレビュー範囲(どこを見れば良いかの範囲)とdoc_15との整合性です。

バックポートは 15と14で英語が同じで日本語訳が違っているもの(修正更新がおこなわれたもの)を抽出して14に適用しています。
そうするとオーバー、ワーカー、スーパーの変更もバックポートされますが、
15で英語が変更されたものや削除された文がオーバ、ワーカ、スーパで残ってしまう場合があります。
そうすると整合が取れないため合わせるために追加で置き換え修正しています。
内容によって適用するしないを判断する(一行で両方あったら一方だけ適用)のは困難なので、
後で一括変更して整合を取る方が簡単です。

Copy link
Contributor

@KenichiroTanaka KenichiroTanaka left a comment

Choose a reason for hiding this comment

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

バックポートなので内容に問題はないはずですが、ざっと確認しました。
問題なさそうです。

@KenichiroTanaka KenichiroTanaka merged commit 6c4a248 into pgsql-jp:doc_ja_14 Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants