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] Pharの翻訳 #230

Merged
merged 4 commits into from
Dec 27, 2024
Merged

[PHP8.4] Pharの翻訳 #230

merged 4 commits into from
Dec 27, 2024

Conversation

natsuki-engr
Copy link
Contributor

No description provided.

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.

ありがとうございます!

訳はそのまま頂きました。それ以外の箇所で修正点があったのでこちらで対応しました:

  • dcf5233: EN-RevisionmasterブランチのコミットIDへ修正
    • masterブランチとは別のコミットID(おそらくプルリクエスト内のコミット)を指していたので、正式なIDであるmasterのそれへ修正しました。
  • 6a929b9: 原文側のタグ変更を翻訳側にも反映
    • 訳の変更といっしょに行われていたタグの変更(<para> -> <simpara>)が反映されていなかったので修正しました。
    • FYI: <para> は昔のタグで、順次 <simpara> に差し替えているようです。

ありがとうございました!

@KentarouTakeda KentarouTakeda merged commit 8b18790 into php:master Dec 27, 2024
1 check passed
@natsuki-engr natsuki-engr deleted the php84-phar branch December 27, 2024 07:10
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