Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Your guide to the Rails Girls Guide!を翻訳した #775

Conversation

maimux2x
Copy link
Collaborator

@maimux2x maimux2x commented Dec 16, 2023

やったこと

https://railsgirls.jp/guide-to-the-guide#1_create_the_application を翻訳しました。

FireShot Capture 052 - Rails Girls - Japanese - localhost

close #773

@maimux2x maimux2x force-pushed the translate_your_guide_to_the_rails_girls_guide branch from 6fc7778 to c2f1e85 Compare December 17, 2023 06:06
@maimux2x maimux2x marked this pull request as ready for review December 17, 2023 06:08
@urimaro urimaro mentioned this pull request Dec 24, 2023
5 tasks
Copy link
Collaborator

@urimaro urimaro left a comment

Choose a reason for hiding this comment

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

翻訳してくださり、ありがとうございます!👏
内部の説明が含まれているので、伝わるように訳すのが難しかったと思います


たくさんコメントしてしまいましたが、nitsimo は気になったところで、must は直したほうが良いと思ったところです
気になり具合の程度は以下のとおりです

  • must > nits > imo

must 以外は、対応する/しないは @maimux2x さんの判断に任せます 🙏

_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Show resolved Hide resolved
_pages/guide-to-the-guide.md Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
@maimux2x
Copy link
Collaborator Author

@urimaro
レビューをいただいてから間が空いてしまい、すみません🙇🏻‍♀️
内容を確認し、修正を反映いたしました。
改めてご確認をいただけると幸いです。

Copy link
Collaborator

@urimaro urimaro left a comment

Choose a reason for hiding this comment

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

@maimux2x
確認に時間がかかってしまってすみません 🙇
1つ1つ丁寧に対応していただいて、ありがとうございます
更新された部分のみ確認しました
今までのコメントでわからないこと、気になることがあれば、後からでも構わないので遠慮なく質問してください 🙏


以下のコメント、素晴らしいと思いました

<!-- Rails Girlsガイドでは多言語対応について解説されていないため、英語表記の例のままにしています -->

まさに t-wada さんの「コードコメントには Why notを書こう」ですね!
https://x.com/t_wada/status/904916106153828352?s=20


改行してしまっている箇所を見つけたので、コメントしました
確認をお願いします 🙏
修正したら merge してしまって 👌 です! :shipit:

_pages/guide-to-the-guide.md Outdated Show resolved Hide resolved
@maimux2x maimux2x merged commit 153f551 into railsgirls-jp:master Jan 22, 2024
2 checks passed
@maimux2x
Copy link
Collaborator Author

@urimaro
こちらこそ毎回丁寧にレビューいただき、ありがとうございました🙏✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Your guide to the Rails Girls Guide!を翻訳する
2 participants