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

[PHP8.4] Change PHP_DEBUG and PHP_ZTS to bool の翻訳 #182

Merged

Conversation

asumikam
Copy link
Contributor

@asumikam asumikam commented Nov 22, 2024

refs #150
php/doc-en#3078 の翻訳作業をしました🍥

@asumikam asumikam force-pushed the Change-PHP_DEBUG-and-PHP_ZTS-to-bool branch from cc5d547 to 7c78c48 Compare November 22, 2024 14:31
@asumikam asumikam changed the title [8.4] Change PHP_DEBUG and PHP_ZTS to bool [8.4] Change PHP_DEBUG and PHP_ZTS to bool の翻訳 Nov 22, 2024
@asumikam asumikam marked this pull request as ready for review November 22, 2024 14:32
@asumikam asumikam changed the title [8.4] Change PHP_DEBUG and PHP_ZTS to bool の翻訳 [PHP8.4] Change PHP_DEBUG and PHP_ZTS to bool の翻訳 Nov 22, 2024
@asumikam asumikam force-pushed the Change-PHP_DEBUG-and-PHP_ZTS-to-bool branch from 7c78c48 to 51ccf25 Compare November 22, 2024 14:33
@KentarouTakeda KentarouTakeda added the review wanted Review wanted. Please feel free to comment; anyone is welcome! label Nov 22, 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.

ありがとうございます!

@@ -108,7 +108,8 @@
<varlistentry xml:id="constant.php-zts">
<term>
<constant>PHP_ZTS</constant>
(<type>int</type>)
(<type>bool</type>)
<constant>ZEND_THREAD_SAFE</constant> &Alias;
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM!

これナイスです!文章の構造のため、原文と順序が逆になるんですね。レビューで初めて知りました…
私が訳してたら、半端に慣れてるおかげでむしろ間違えていたかもしれない😇

Copy link
Contributor Author

Choose a reason for hiding this comment

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

私もプレビューで見て、「なんか・・・これ変だな・・・」ってなって気づきました!ww
ありがとうございます!!!

@KentarouTakeda KentarouTakeda merged commit f65555f into php:master Nov 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review wanted Review wanted. Please feel free to comment; anyone is welcome!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants