Skip to content

Conversation

@kaito3desuyo
Copy link
Contributor

翻訳・修正

  • [✓] 変更内容はCONTRIBUTING.md に記載されたワークフローに従っています

関連Issue

#887

備考

assertの訳でしっくり来るものがなかったため、一旦カタカナでアサートにしてあります。
また、「インジェクションコンテキスト内で実行する」節の説明文訳がちょっと怪しいので、実際のrunInInjectionContextの役割と比して乖離していないかを査読いただきたいです。

Copy link
Collaborator

@lacolaco lacolaco left a comment

Choose a reason for hiding this comment

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

翻訳ありがとうございます!

@lacolaco
Copy link
Collaborator

@kaito3desuyo
アサートの部分はこのままマージしてもよいかと思いますが、変更の提案はしましたので、取り込むかどうか判断していただきたいです。取り込まない場合も教えてください。

@kaito3desuyo
Copy link
Contributor Author

@lacolaco
ありがとうございます。
「検証」については私も候補として考えておりましたので、変更させていただきたいと思います。
取り込みますのでよろしくお願いいたします。

アサート→検証

Co-authored-by: Suguru Inatomi <suguru.inatomi@gmail.com>
@lacolaco
Copy link
Collaborator

ありがとうございます!マージさせていただきます

@lacolaco lacolaco linked an issue Oct 21, 2023 that may be closed by this pull request
@lacolaco lacolaco merged commit e6d6c2b into angular:main Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

guide/dependency-injection-context の翻訳

2 participants