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

Conversation

@YouheiNozaki
Copy link
Contributor

概要

docs/tutorial/gatsby-image-tutorialの翻訳を完了しました。
遅くなってしまい、大変申し訳ありません。レビューよろしくお願い致します。

チェックリスト

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

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

補足

Ref: #1

@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Mar 3, 2020
Copy link
Contributor

@uetchy uetchy left a comment

Choose a reason for hiding this comment

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

👍

@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 Mar 4, 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.

@YouheiNozaki PR ありがとうございます!本家の方に更新が入ったようで、この PR と Conflict しているので、Conflict の解消をお願いできますでしょうか。

@YouheiNozaki
Copy link
Contributor Author

Conflict対処いたしました。よろしくお願い致します。
その他の修正も早急に行います。遅くなってしまっていて申し訳ないです。。少々お持ちください。

@YouheiNozaki
Copy link
Contributor Author

その他のsuggestも修正しました。ご確認よろしくお願い致します。

@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
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.

コンフリクトの解消ありがとうございます!
いくつか suggest したので、ご確認お願いします。

@Naturalclar Naturalclar 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 8, 2020
@YouheiNozaki
Copy link
Contributor Author

ありがとうございます!
修正多くて申し訳ないです。。。
修正いたしましたので、ご確認よろしくお願い致します。
よろしければ54行目のNoteの使い方が統一されていない件、自分がPR送ってみてもよろしいですか?

@Naturalclar
Copy link
Contributor

@YouheiNozaki

よろしければ54行目のNoteの使い方が統一されていない件、自分が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.

ありがとうございます!LGTM

@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 Mar 9, 2020
Copy link
Contributor

@uetchy uetchy left a comment

Choose a reason for hiding this comment

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

🎉

@uetchy uetchy merged commit e71ec42 into gatsbyjs:master Mar 9, 2020
@gatsbot
Copy link

gatsbot bot commented Mar 9, 2020

Holy buckets, @YouheiNozaki — 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 @YouheiNozaki!

@YouheiNozaki
Copy link
Contributor Author

こちらこそ修正ありがとうございました!
多くて申し訳なかったです。そして、コントリビュートさせていただける機会をくださってありがとうございました!
今後はこの経験を生かして、自分に出来るコントリビュートを積極的にしていきます!
早速↑のPR送ってみます!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants