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

fixes #324 hooks/advanced-topics/index.md typoの修正 #325

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stein2nd
Copy link

@stein2nd stein2nd commented Nov 5, 2024

Fixes #324

原文 125 行目:  did_action()
訳文 127 行目: did_action()

@t-hamano
Copy link
Member

t-hamano commented Nov 6, 2024

ご対応ありがとうございます。こちら、原文リポジトリにもPRを提出していただくことは可能でしょうか。

@stein2nd
Copy link
Author

stein2nd commented Nov 6, 2024

明日以降にでも、原文側に PR 出してみます。

@t-hamano
Copy link
Member

t-hamano commented Nov 8, 2024

原文側にもPRを立てましたら、その PR 番号をここにリンクしていただけると助かります。

@stein2nd
Copy link
Author

stein2nd commented Nov 9, 2024

下記の様に、issue を立てて。
WordPress#38

下記の様に、修正 PR を投げました(レビュー&マージ待ち)。
WordPress#39

image
↑を↓の様に、修正する…という簡単な内容です。
image

@stein2nd
Copy link
Author

stein2nd commented Nov 25, 2024

こちら、原文側mainにmaergeされました。

@t-hamano
Copy link
Member

t-hamano commented Nov 26, 2024

ありがとうございます!

原文側の別のPR( WordPress#42 )がマージされ次第、日本語リポジトリもUpstreamに追従させ、このPRを進められるようにしたいと思います。

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.

hooks/advanced-topics/index.md typoの修正
2 participants