Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Conversation

@chanrute
Copy link
Contributor

@chanrute chanrute commented Mar 7, 2020

概要

  • issue 表記揺れの修正 #207 「表記揺れの修正」の対応PR
  • ドキュメント内の”リスト”という文字列を”一覧”に変更する
  • 今後の翻訳に対しても表記ゆれが起きないよう、prh.yml にルールを追加した

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました。
  • 文章全体を最初から読み直して不自然な箇所が無いことを確認しました。
  • Allow edits from maintainers にチェックを入れました。

しばらく待ってもレビューが終わらなかったり、必要なレビュー数に足りない状態が続いた場合は、こちらからメンテナーを探して、@を付けてメンションを飛ばしてください。

補足

以前、#100 で翻訳Contributeを行わせていただきました。
少し時間が空いたので、翻訳の方もお手伝いしたかったのですが、承認をいただくまでに時間がかかりそうだったので、issuesの対応をさせて頂きました(不要な対応であれば削除頂いてもかまいません)。

Ref: #207

chanrute added 2 commits March 7, 2020 19:16
Add expect and pattern rules in prh.yml to correct
orthographic variants in issues207.
To do with  issues 207, convert "リスト" to "一覧" at once.
@chanrute chanrute linked an issue Mar 7, 2020 that may be closed by this pull request
@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Mar 7, 2020
@smorimoto smorimoto added status: awaiting author response A pull request that is awaiting author's response and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Mar 7, 2020
Unnecessary conversion was corrected.
And change rule setting method in prh.yml.
@chanrute
Copy link
Contributor Author

chanrute commented Mar 7, 2020

@imbsky
レビューおよびsuggestionを頂き、ありがとうございます。
修正をさせて頂きました。

@smorimoto
Copy link
Contributor

@f-teruhisa 修正ありがとうございます。今回は良いので、次回からサジェストをコミットする際は #202 (comment) にあるようにしてください。

@smorimoto smorimoto added status: awaiting review A pull request that is currently awaiting a reviewer's response and removed status: awaiting author response A pull request that is awaiting author's response labels Mar 7, 2020
@chanrute
Copy link
Contributor Author

chanrute commented Mar 7, 2020

ありがとうございます。了解しました。
次回より、サジェストはPR上で直接取り込むようにします。

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

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

ありがとうございます!

@Naturalclar
Copy link
Contributor

Closes #207

@Naturalclar Naturalclar added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Mar 8, 2020
@uetchy uetchy added status: ready to merge A pull request that can be merged once CI is green and removed status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned labels Mar 8, 2020
@smorimoto smorimoto merged commit 05de2ba into gatsbyjs:master Mar 8, 2020
@chanrute chanrute deleted the issues207/fix_orthographic_variants branch March 10, 2020 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: ready to merge A pull request that can be merged once CI is green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

表記揺れの修正

4 participants