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

Conversation

@lilpacy
Copy link
Contributor

@lilpacy lilpacy commented Jan 18, 2020

概要

docs/third-party-graphqlの日本語翻訳をしました。

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました(推奨)
  • Allow edits from maintainers にチェックを入れました。

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

@lilpacy lilpacy force-pushed the docs/third-party-graphql branch from 34dc58d to 7bd3438 Compare January 18, 2020 17:22
@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 18, 2020

レビューよろしくお願いいたします。
オリジナル:Using Third-party GraphQL APIs

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

lilpacy commented Jan 19, 2020

早速のレビューありがとうございます。
半角→全角、あるいはその逆でスペースを入れられてなかった点、提案いただいた通り修正いたしました。

@smorimoto
Copy link
Contributor

@lilpacy サジェストの適応ありがとうございます。一箇所だけ追加でサジェストを入れました。

スペース関連のチェックだけざっとしたのですが、中身までは時間がなくて見れてないので他のメンテナーが確認可能であればお願いします。

@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 19, 2020

若干修正しました。

@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 24, 2020

リマインドです、よろしくお願いいたします!
@Naturalclar
@hirotaka
@uetchy

uetchy
uetchy previously approved these changes Jan 24, 2020
@uetchy uetchy 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 Jan 24, 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 したのでご確認お願いします。

lilpacy and others added 2 commits January 25, 2020 16:17
@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 25, 2020

@Naturalclar Suggestありがとうございました!
Suggestいただいた内容でコミットさせていただきました。ご確認よろしくお願いします。

@uetchy
Copy link
Contributor

uetchy commented Jan 25, 2020

@lilpacy diffに表示されているtextlintエラーを解消してください👍

@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 25, 2020

@uetchy ご指摘ありがとうございます。算用数字へと修正いたしました。
よろしくお願いいたします。

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!ありがとうございます!

@lilpacy
Copy link
Contributor Author

lilpacy commented Jan 26, 2020

@hirotaka 一応、もう一度リマインドです、よろしくお願いいたします。
お忙しい中すみません。

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 8f38903 into gatsbyjs:master Jan 26, 2020
@gatsbot
Copy link

gatsbot bot commented Jan 26, 2020

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

@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Jan 26, 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