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

[PHP 8.4] fix php/doc-en#4122 (PHP 8.4: Flushing headers without a body will now succeed) #188

Merged
merged 4 commits into from
Nov 24, 2024

Conversation

o0h
Copy link
Contributor

@o0h o0h commented Nov 24, 2024

Issue

元ネタ: php/doc-en#4122
ref: #150

該当ファイルの最新コミットが、このマージコミットになっているので
そのまま4122のFile Changedとの対応をとればOK

image

翻訳の内容は、 appendices/migration84/other-changes.xml を参照(尊重)しています

<sect3 xml:id="migration84.other-changes.sapi.fpm">
<title>FPM</title>
<simpara>
本文なしでヘッダをフラッシュできるようになりました。
<!-- See GH-12785. -->
</simpara>

オリジナル(en)でも、

  • migration: Flushing headers without a body will now succeed.
  • reference: Flushing headers without a body will now succeed in FastCGI.

となっているので、「殆どそのまま持ってくる」のが無難そうだと判断しました。

Refs

Preview

image

@o0h o0h marked this pull request as ready for review November 24, 2024 04:40
@o0h o0h changed the title [WIP] fix php/doc-en#4122 (PHP 8.4: Flushing headers without a body will now succeed) fix php/doc-en#4122 (PHP 8.4: Flushing headers without a body will now succeed) Nov 24, 2024
@o0h o0h changed the title fix php/doc-en#4122 (PHP 8.4: Flushing headers without a body will now succeed) [PHP 8.4] fix php/doc-en#4122 (PHP 8.4: Flushing headers without a body will now succeed) Nov 24, 2024
Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

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

ありがとうございます!
Thank you for your first contribution🎉

1コミットだけ追加させて頂きました。
ファイル冒頭のリビジョン番号を原文と合わせておくことで、原文が更新されていた場合ツールでそれを検知する、と言ったことが行えます。もし余裕があればチェック頂けると助かります。(こちらで更新するのでも大丈夫です)

@KentarouTakeda KentarouTakeda merged commit 9a079f4 into php:master Nov 24, 2024
1 check passed
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="utf-8"?>
<!-- $Revision$ -->
<!-- EN-Revision: 91ab4f5f898023b0eed0e642e1482ac11f749d20 Maintainer: hirokawa Status: ready -->
<!-- EN-Revision: bac9d6a54fae363b3cc337bda924a76ff47e8851 Maintainer: hirokawa Status: ready -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@KentarouTakeda (後学のために質問させてください!

ココに記載するのって、reference/outcontrol/functions/flush.xml の最終コミットかな?と思っていたのですが
php/doc-en@77ae333
ではなくて
php/doc-en@bac9d6a
であってますか? 💡

cf: 手元で生成した revcheck.html のキャプ
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

すみませんでした!はい、合ってます、つまり私の修正ミスです。

修正しました。非常に助かりました!!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます、完璧に覚えました!
次回からは自分の作業に含められそうです 👍

@o0h o0h deleted the flush-84 branch November 24, 2024 14:52
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