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

Conversation

@hoshikitsunoda
Copy link
Contributor

概要

Translation for: https://www.gatsbyjs.org/docs/themes/getting-started/

チェックリスト

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

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

@hoshikitsunoda hoshikitsunoda changed the title Docs/themes/getting started Translattion: docs/themes/getting-started Jan 17, 2020
@hoshikitsunoda hoshikitsunoda changed the title Translattion: docs/themes/getting-started Translation: docs/themes/getting-started Jan 17, 2020
@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Jan 18, 2020
package.json Outdated
]
},

"version": "0.0.0",
Copy link
Contributor

@uetchy uetchy Jan 24, 2020

Choose a reason for hiding this comment

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

翻訳以外の変更を含める場合は、その部分を切り出して新たなPRを送ってください。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

訂正ありがとうございます!部分的に切り出して新たなPRを送るということをしたことがないのですが、一旦この修正前までrevertして新しいブランチを作ってからPRを送るんですか?

Copy link
Contributor

Choose a reason for hiding this comment

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

選択的に変更を削除して新しいヒストリーを作り直し、それをforce pushする形になりますね。
このPRには翻訳部分のみを含む形にします。
もし難しければこちらでやるので言ってください。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

遅くなりました。今package.jsonを2commit前に戻しインラインコードを修正してpushしたんですがこれであってますか?まだfile changedが2になっているのでpackage.jsonは完全に消さないといけないですか?

Copy link
Contributor

Choose a reason for hiding this comment

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

package.json への変更は含めないようにお願いします!

### 1. テーマをインストールし設定する

When you use a starter that's built with a theme, you will often see that you're initially presented with a lighter weight `gatsby-config.js`. Themes start doing their magic when installed via the `plugins` array:
テーマを使用して作られたスターターの`gatsby-config.js`は最初軽く見えることが多くあります。テーマはプラグイン配列からインストールされたときに真価を発揮します。
Copy link
Contributor

Choose a reason for hiding this comment

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

plugins のようなインラインコードは翻訳しないでください。

@uetchy uetchy added the status: awaiting author response A pull request that is awaiting author's response label Jan 25, 2020
@smorimoto smorimoto removed the status: awaiting author response A pull request that is awaiting author's response label Jan 26, 2020
@smorimoto smorimoto requested a review from uetchy January 26, 2020 14:17
@@ -1,42 +1,34 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

package.jsonが古いものにロールバックされている状態です。自力で解決するのが困難であれば教えてください。こちらで修正します。

Copy link
Contributor Author

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 status: awaiting review A pull request that is currently awaiting a reviewer's response and removed status: awaiting review A pull request that is currently awaiting a reviewer's response status: awaiting author response A pull request that is awaiting author's response labels Jan 26, 2020
@uetchy
Copy link
Contributor

uetchy commented Jan 27, 2020

@hoshikitsunoda package.jsonに対するコミットをドロップして、ついでに最新のmasterにrebaseしました👍

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 Jan 27, 2020
@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 Jan 27, 2020
@hoshikitsunoda
Copy link
Contributor Author

ありがとうございます!お手数おかけしました。。さきほどsuggestionをcommitしました!
よければでいいのですが、どういうコマンドを入力してコミットのドロップとrebaseをしたのか教えてもらってもいいですか?

@uetchy
Copy link
Contributor

uetchy commented Jan 27, 2020

@hoshikitsunoda Forkを使ってinteractive rebaseを行いました👍

uetchy
uetchy previously approved these changes Jan 27, 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 27, 2020
Copy link
Contributor

@hirotaka hirotaka left a comment

Choose a reason for hiding this comment

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

内容は大丈夫なのですが、体裁に関するところだけ、いくつかSuggestionをリクエストさせていただいのでご確認お願いします。

@hoshikitsunoda hoshikitsunoda dismissed stale reviews from uetchy via db19c38 January 28, 2020 10:44
@hoshikitsunoda
Copy link
Contributor Author

内容は大丈夫なのですが、体裁に関するところだけ、いくつかSuggestionをリクエストさせていただいのでご確認お願いします。

見逃してました。ありがとうございます。suggestion commitさせていただきました。よろしくお願いします!

Copy link
Contributor

@hirotaka hirotaka left a comment

Choose a reason for hiding this comment

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

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

@hirotaka hirotaka requested a review from uetchy January 28, 2020 10:47
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 a006151 into gatsbyjs:master Jan 28, 2020
@gatsbot
Copy link

gatsbot bot commented Jan 28, 2020

Holy buckets, @hoshikitsunoda — 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 28, 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