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

Conversation

@blkclct
Copy link
Contributor

@blkclct blkclct commented Feb 12, 2020

概要

I translated docs/adding-react-components page.
The page is docs/adding-react-components.
I appreciate to reviews to my pull request.

チェックリスト

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

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

補足

Ref: #1

@uetchy uetchy added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Feb 12, 2020
@blkclct
Copy link
Contributor Author

blkclct commented Feb 13, 2020

5b6ccda でご指摘いただいていた箇所を修正しました.

@blkclct blkclct force-pushed the docs/adding-react-components branch from fe69136 to 50f6519 Compare February 13, 2020 15:31
@blkclct
Copy link
Contributor Author

blkclct commented Feb 13, 2020

ced8c1c, 77023bf, 3ae8714 で指摘をいただいていた箇所を修正しました.

@uetchy uetchy 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 Feb 15, 2020
Naturalclar
Naturalclar previously approved these changes Feb 15, 2020
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.

読みやすい訳と感じました!
原文が少しだけUpdateされたようで、Conflictしているのでその解消をお願いできますでしょうか?

@blkclct
Copy link
Contributor Author

blkclct commented Feb 15, 2020

@Naturalclar
お忙しい中,レビューありがとうございます!
原文がアップデートされたこと確認いたしまして,更新します!

一点質問させてください🙇‍♂️
masterブランチをマージすると他の差分も入ってしまうようです.例えば,未翻訳のページでpull request Pull Requestのように変更された差分が混入してしまいます.
このような差分は除外した形でマージしてpushするというオペレーションで間違いありませんか??

@Naturalclar
Copy link
Contributor

@blkclct

masterブランチをマージすると他の差分も入ってしまうようです.未翻訳のページでpull request がPull Requestのように変更された差分が混入してしまいます.
このような差分は除外した形でマージしてpushするというオペレーションで間違いありませんか??

はい、担当のファイル以外の差分は除外してPushしていただければ大丈夫です!

@blkclct
Copy link
Contributor Author

blkclct commented Feb 15, 2020

@Naturalclar
ご回答ありがとうございます!そのように対応いたします.

Naturalclar
Naturalclar previously approved these changes Feb 15, 2020
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.

ありがとうございます!:+1:

@Naturalclar Naturalclar added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting author response A pull request that is awaiting author's response labels Feb 15, 2020
@blkclct
Copy link
Contributor Author

blkclct commented Feb 15, 2020

コンフリクトの解消とご指摘いただいていた箇所を変更しました!

再度レビューお願いします 🙇‍♂️

@Naturalclar さん
すみません.足りてない翻訳箇所がありましたので 2a4170e で追記しました 🙇‍♂️

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.

👍

@smorimoto smorimoto merged commit e3faaf6 into gatsbyjs:master Feb 16, 2020
@gatsbot
Copy link

gatsbot bot commented Feb 16, 2020

Holy buckets, @blkclct — we just merged your PR to Gatsby! 💪💜

Gatsby is built by awesome people like you. Let us say “thanks” in two ways:

  1. We’d like to send you some Gatsby swag. As a token of our appreciation, you can go to the Gatsby Swag Store and log in with your GitHub account to get a coupon code good for one free piece of swag. We’ve got Gatsby t-shirts, stickers, hats, scrunchies, and much more. (You can also unlock even more free swag with 5 contributions — wink wink nudge nudge.) See gatsby.dev/swag for details.
  2. We just invited you to join the Gatsby organization on GitHub. This will add you to our team of maintainers. Accept the invite by visiting https://github.com/orgs/gatsbyjs/invitation. By joining the team, you’ll be able to label issues, review pull requests, and merge approved pull requests.

If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’.

Thanks again!

@smorimoto
Copy link
Contributor

Thanks @blkclct for your contribution!

@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Feb 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants